[GitHub] twill pull request #62: (TWILL-248) Upgrade to use Netty-4.1

[chtyim]

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

    https://github.com/apache/twill/pull/62#discussion_r143854276
  
    --- Diff: 
twill-yarn/src/main/java/org/apache/twill/yarn/ResourceReportClient.java ---
    @@ -52,12 +54,16 @@
       public ResourceReport get() {
         for (URL url : resourceUrls) {
           try {
    -        Reader reader = new BufferedReader(new 
InputStreamReader(url.openStream(), Charsets.UTF_8));
    -        try {
    +        HttpURLConnection urlConn = (HttpURLConnection) 
url.openConnection();
    --- End diff --
    
    Not entirely. This is to enable compression when fetching the report.


---