kangkaisen commented on a change in pull request #3755:
URL: https://github.com/apache/incubator-doris/pull/3755#discussion_r434305139



##########
File path: fe/src/main/java/org/apache/doris/plugin/PluginZip.java
##########
@@ -113,13 +113,13 @@ Path downloadRemoteZip(Path targetPath) throws 
IOException, UserException {
         cleanPathList.add(zip);
 
         // download zip
-        try (InputStream in = openUrlInputStream(source)) {
+        try (InputStream in = Util.getInputStreamFromUrl(source, null, 5000, 
5000)) {

Review comment:
       Should be static const value.
   If the timeout isn't a config, I think set a larger value maybe better when 
the plugin resource is big and network is poor.




----------------------------------------------------------------
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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to