-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Increase the timeout to prevent timeouts when uploading a large directory #99
Conversation
…actConfig.java Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
…actConfig.java Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
…irtualFile.java Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
…irtualFile.java Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
…actConfig.java Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
…ArtifactConfig/help-disableExternalUrl.html Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
…s when uploading a large directory (over 10,000 files).
@@ -245,7 +245,7 @@ public String getUrl() { | |||
private static class UploadToBlobStorage extends MasterToSlaveFileCallable<Void> { | |||
|
|||
public static final int MAX_QUEUE_SIZE_IN_NETTY = 500; | |||
public static final int TIMEOUT = 30; | |||
public static final int TIMEOUT = 600; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
have you confirmed this works in your case? should this be configurable to balance between not hanging jobs and larger uploads?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, it is working good for my usecase.
should this be configurable to balance between not hanging jobs and larger uploads?
how about your opinion?
and I don't think it will hanging jobs because the timeout is still there.
if larger uploads taking too much time, I believe users should consider compressing files into a zip format before uploading them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets try it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure, thank you.
for fix #95
Increase the timeout from 30 seconds to 10 minutes to prevent timeouts when uploading a large directory (contains over 10,000 files).
Testing done