6/12/2017 2:32:25 PM
warning (#1215)
http://errorprone.info/bugpattern/ImmutableEnumChecker
|
|
|
6/12/2017 12:40:43 AM
even if test runner is slow (like supposedly on Travis sometimes)
This doesn't make the tests any slower than before, as long as they pass
Why would this help? Even locally the FlowRunner tests started failing when I put 10 instead of 100 max tries (10 ms wait per try). 1000 should make it pretty robust :)
For background, see:
PR #1179 (comment)
https://travis-ci.org/azkaban/azkaban/builds/240580566?utm_source=github_status&utm_medium=notification
PR #1201 (comment)
PR #1186
|
|
|
6/12/2017 12:06:02 AM
sleep for 20 seconds and the comments are wrong.
When debugging the test, the test would fail since it is time sensitive.
Fix:
Switch to use Jodatime utility to get the current time and manipulate the time in the test.
It's better to use the java time api and use a Guice injected wrapper class to manipulate time. However given that BasicTimeChecker is scheduled to be deprecated soon. It's not worth investing in it now.
|
6/11/2017 8:51:01 PM
handling related changes (#1189)
Initial motivation was to remove ImmutableEnumChecker warnings. Worked with @kunkun-tang to determine that Guicing the offending classes was the best way to proceed.
MetricsManager, CommonMetrics, and WebMetrics had to be guicified together in order to handle tests that used both CommonMetrics/MetricsManager or WebMetrics/MetricsManager.
A number of tests required minor refactoring to handle the new guicier versions.
Certain classes were only partially guiced in order to minimize the size of this PR. TODOs were added so these classes could be followed up on and fully guiced in the future.
I'm hoping these testing changes/guicing overall will allow for more parallel testing. Additional adjustment of testing classes is likely necessary to make that happen.
|
6/10/2017 8:38:31 PM
is no more warning in this file after this change.
|
6/10/2017 8:27:00 PM
fixed a logger error which logs only the exception message
instead of the complete call stack.
Warnings fixed:
from gradle:
Note: /Users/ruyang/oss/azkaban/azkaban-exec-server/src/main/java/azkaban/execapp/AzkabanExecutorServer.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
compiler:
Warning:(294, 17) java: propagate(java.lang.Throwable) in com.google.common.base.Throwables has been deprecated
Warning:(306, 17) java: propagate(java.lang.Throwable) in com.google.common.base.Throwables has been deprecated
|
|
|
6/9/2017 11:14:36 AM
(#1190)
This is to promote consistency.
|
