Skip to content

Conversation

ARibster
Copy link
Contributor

What problem are we solving?

Issue #6746: S3 Feature: please add s3.idleTimeout command line parameter
( #6746 )

How are we solving the problem?

Following the command line parameter scheme, I have adde "idleTimeout" parameter to the S3 standalone and "s3.idleTimeout" to the server and filer options. I have kept the default value 10

How is the PR tested?

weed s3 without idleTimeout parameter:
time telnet 127.0.0.1 8333
Connected to 127.0.0.1.
telnet: Connection closed by foreign host
real 10.030
user 0.15000
sys 0.000
weed s3 with -idleTimeout=20:
time telnet 127.0.0.1 8333
Connected to 127.0.0.1.
telnet: Connection closed by foreign host
real 20.033
user 0.000
sys 0.000
weed server with filer and s3 API without idleTimeout parameter:
time telnet 127.0.0.1 8333
Connected to 127.0.0.1.
telnet: Connection closed by foreign host
real 10.031
user 0.000
sys 0.30000
weed server with filer and s3 API with -s3.idleTime=30:
time telnet 127.0.0.1 8333
Connected to 127.0.0.1.
telnet: Connection closed by foreign host
real 30.034
user 0.000
sys 0.61000
weed filer with s3 API without idleTimeout parameter:

time telnet 127.0.0.1 8333
Connected to 127.0.0.1.
telnet: Connection closed by foreign host
real 10.034
user 0.000
sys 0.15000
weed filer with s3 API with -s3.idleTimeout=30:
time telnet 127.0.0.1 8333
Connected to 127.0.0.1.
telnet: Connection closed by foreign host
real 30.032
user 0.15000
sys 0.46000

Checks

  • I have added unit tests if possible.
  • I will add related wiki document changes and link to this PR after merging.

@chrislusf chrislusf merged commit d424cbb into seaweedfs:master May 1, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants