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.
|
6/8/2017 2:05:36 PM
warning: [ReferenceEquality] Comparison using reference equality instead of value equality
begin == "" ? -1 : DateTimeFormat.forPattern(FILTER_BY_DATE_PATTERN)
^
(see http://errorprone.info/bugpattern/ReferenceEquality)
Did you mean '"".equals(begin) ? -1 :
"
Also see
https://stackoverflow.com/questions/7520432/what-is-the-difference-between-vs-equals-in-java
|