[GitHub] twill pull request #65: (TWILL-254) Update to use ContainerId.fromString

[serranom]

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

    https://github.com/apache/twill/pull/65#discussion_r165965620
  
    --- 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 --
    
    Ah, okay then.  sorry, I missed that.


---