-
Notifications
You must be signed in to change notification settings - Fork 193
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
add tracking for checksum algorithm and retry mode business metrics #3944
Conversation
A new generated diff is ready to view.
A new doc preview is ready to view. |
UA metrics for checksums is covered in #3851 and it sitting in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@@ -130,6 +131,22 @@ where | |||
cfg: &mut ConfigBag, | |||
) -> Result<(), BoxError> { | |||
let checksum_algorithm = (self.algorithm_provider)(context.input())?; | |||
if let Some(checksum_algorithm) = checksum_algorithm { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would probably remove these changes for now and just roll out this detection with the new flex checksums (mostly in favor of making my merge to the feature branch easier)
A new generated diff is ready to view.
A new doc preview is ready to view. |
A new generated diff is ready to view.
A new doc preview is ready to view. |
Adds business metric tracking for checksum algorithms and retry modes. This PR includes tests.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.