Skip to content

Commit

Permalink
Add client execution timeout
Browse files Browse the repository at this point in the history
  • Loading branch information
eheinlein-verta committed Jul 26, 2023
1 parent 3f8cfc2 commit d0ee082
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@ public S3Client(S3Config s3Config) throws IOException, ModelDBException {
this.defaultClientConfig =
new ClientConfiguration()
.withMaxConnections(s3Config.getConnectionPoolSize())
.withRequestTimeout(s3Config.getRequestTimeoutMs());
.withRequestTimeout(s3Config.getRequestTimeoutMs())
.withClientExecutionTimeout(s3Config.getClientExecutionTimeout());

// Start the counter with one because this class has a reference to it
referenceCounter = new AtomicInteger(1);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ public class S3Config {
@JsonProperty private String awsRegion = "us-east-1";
@JsonProperty private Boolean s3presignedURLEnabled = true;
@JsonProperty private Integer requestTimeoutMs = 30 * 1000;
@JsonProperty private Integer clientExecutionTimeout = 40 * 1000;
@JsonProperty private Integer connectionPoolSize = 75;

public void validate(String base) throws InvalidConfigException {
Expand Down

1 comment on commit d0ee082

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docker Tag: eh_VUMM-1639-s3-config-2023_03-2023-07-26T18-31-07--d0ee082

Please sign in to comment.