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

    https://github.com/apache/twill/pull/65#discussion_r165862926
  
    --- 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 --
    
    If marked `final`, I can not extend it for 
[`Hadoop26YarnAMClient`](https://github.com/apache/twill/pull/65/files#diff-8c254de8ae00c6007495979dcb66a986R30)?


---

Reply via email to