[GitHub] [hadoop-ozone] vivekratnavel commented on a change in pull request #783: HDDS-2976. Recon throws error while trying to get snapshot over https

[GitBox <gi...@apache.org>]

vivekratnavel commented on a change in pull request #783: HDDS-2976. Recon throws error while trying to get snapshot over https
URL: https://github.com/apache/hadoop-ozone/pull/783#discussion_r405065739
 
 

 ##########
 File path: hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/ReconUtils.java
 ##########
 @@ -245,6 +246,16 @@ public InputStream makeHttpCall(CloseableHttpClient httpClient, String url)
     }
   }
 
+  public InputStream makeHttpCall(URLConnectionFactory connectionFactory,
+                                  String url)
+      throws IOException {
+
+    java.net.URLConnection urlConnection =
 
 Review comment:
   NIT: We can import java.net.URLConnection and java.net.URL

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org