-
Notifications
You must be signed in to change notification settings - Fork 16
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
Upgrade dependencies and fix unit tests #37
Conversation
@@ -598,11 +599,11 @@ private void buildReportWithSleep(final CloudWatchReporter.Builder cloudWatchRep | |||
* | |||
* @throws NoSuchFieldException | |||
* @throws IllegalAccessException | |||
* @see Meter#tickIfNecessary() | |||
* @see ExponentialMovingAverages#tickIfNecessary() | |||
* @see MetricDatum#getDimensions() | |||
*/ | |||
private static void reduceMeterDefaultTickInterval() throws NoSuchFieldException, IllegalAccessException { |
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.
@madanepuri can you please provide some context about why this function has to be updated? Also, if it must to be updated, can you also update its name
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.
Also, please update the docs as you are changing the function behavior and this needs to be clearly communicated <= never mind, this is just a test
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.
@azagniotov TICK_INTERVAL
field is moved to ExponentialMovingAverages class now. Previously it was in Meter class. Updated the function names. Please review. Thanks
47ec8c6
to
9a1dcbc
Compare
93f54b6
to
3f42df8
Compare
3f42df8
to
de54144
Compare
No description provided.