-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(blooms): Add task timining and sizing metrics #15032
Conversation
Hello @salvacorts!
Please, if the current pull request addresses a bug fix, label it with the |
(cherry picked from commit 79cccfd)
What this PR does / why we need it:
This PR adds observability around the time it takes tasks to complete as well as metrics for the amount of data the tasks will process.
We also refactor the planning strategy factory from a function into a struct. This is needed so we don't need to recreate the metrics.
Special notes for your reviewer:
Checklist
CONTRIBUTING.md
guide (required)feat
PRs are unlikely to be accepted unless a case can be made for the feature actually being a bug fix to existing behavior.docs/sources/setup/upgrade/_index.md
deprecated-config.yaml
anddeleted-config.yaml
files respectively in thetools/deprecated-config-checker
directory. Example PR