[GitHub] drill pull request #1099: DRILL-6106: Use valueOf method instead of construc...

[vrozov]

Github user vrozov commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1099#discussion_r164010865
  
    --- Diff: 
contrib/storage-hive/core/src/main/codegen/templates/ObjectInspectors.java ---
    @@ -169,9 +169,9 @@ public Boolean getPrimitiveJavaObject(Object o) {
           if (o == null) {
             return null;
           }
    -      return new Boolean(((NullableBitHolder)o).value == 0 ? false : true);
    +      return Boolean.valueOf(((NullableBitHolder)o).value == 0 ? false : 
true);
    --- End diff --
    
    either `Boolean.valueOf(((NullableBitHolder)o).value != 0);` or 
`((NullableBitHolder)o).value == 0 ? Boolean.FALSE : Boolean.TRUE;`


---