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

    https://github.com/apache/twill/pull/71#discussion_r221663573
  
    --- 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 --
    
    What does the `i` prefix mean? Better just call this `hasDFSUtilClient`.


---

Reply via email to