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
|
|
|
6/8/2017 2:13:21 PM
is the last warning including style check for this file I can find.
Reformatting doesn't seem to correct this problem automatically.
|
