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

    https://github.com/apache/twill/pull/65#discussion_r165845929
  
    --- Diff: 
twill-yarn/src/main/hadoop22/org/apache/twill/internal/yarn/Hadoop22YarnAMClient.java
 ---
    @@ -26,7 +26,7 @@
     /**
      * Wrapper class for AMRMClient for Hadoop version 2.2 or greater.
      */
    -public final class Hadoop22YarnAMClient extends Hadoop21YarnAMClient {
    +public class Hadoop22YarnAMClient extends Hadoop21YarnAMClient {
     
    --- End diff --
    
    why remove final?


---

Reply via email to