[GitHub] twill pull request #71: TWILL-262 YarnUtils#cloneHaNnCredentials uses DFSUti...

[piaosama]

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

    https://github.com/apache/twill/pull/71#discussion_r223181337
  
    --- Diff: 
twill-yarn/src/main/java/org/apache/twill/internal/yarn/YarnUtils.java ---
    @@ -72,7 +77,23 @@
         HADOOP_26
       }
     
    -  private static final Logger LOG = 
LoggerFactory.getLogger(YarnUtils.class);
    +  private static boolean iDFSUtilClientExists = false; // use this to 
judge if the hadoop version is above 2.8
    --- End diff --
    
    ok， suddenly find HAUtil class does not exists as the DFSUtils class. i 
will add the two judge into the code and make the code more nice as you 
commented.


---