chore: Add doc and rename function for flushing strategy by lym953 · Pull Request #740 · DataDog/datadog-lambda-extension
lym953
marked this pull request as ready for review
lym953
deleted the
yiming.luo/flush-strategy-doc
branch
duncanpharvey pushed a commit that referenced this pull request
Mar 10, 2026# Motivation It took me quite some effort to understand flushing strategies. I want to make it easier to understand for me and future developers. # This PR Tries to make flushing strategy code more readable: 1. Add/move comments 2. Create an enum `ConcreteFlushStrategy`, which doesn't contain `Default` because it is required to be resolved to a concrete strategy 3. Rename `should_adapt` to `evaluate_concrete_strategy()` # To reviewers There are still a few things I don't understand, which are marked with `TODO`. Appreciate explanation! Also correct me if any comment I added is wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters