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.
|
|
|
6/8/2017 6:39:28 PM
on assertj library, which has a few advantages.
see
http://randomthoughtsonjavaprogramming.blogspot.com/2017/03/assertj-vs-hamcrest.html
This will speed up the Travis build process a bit.
|
6/8/2017 4:28:44 PM
(#1184)
assertj library has a few advantages.
see
http://randomthoughtsonjavaprogramming.blogspot.com/2017/03/assertj-vs-hamcrest.html
|
|
|
