-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[opt](s3client) Add AWS SDK logger for recycler with error logging and enhance S3 client tests #49422
base: master
Are you sure you want to change the base?
Conversation
run buildall |
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
…d enhance S3 client tests * Add aws sdk logger for recycler * Default set `aws_log_level=2` for logging aws sdk error * Add more ut for `S3ObjStorageClient`
d751e30
to
e17568c
Compare
run buildall |
TeamCity cloud ut coverage result: |
TPC-H: Total hot run time: 34555 ms
|
TPC-DS: Total hot run time: 186412 ms
|
ClickBench: Total hot run time: 31.02 s
|
run buildall |
TeamCity cloud ut coverage result: |
TPC-H: Total hot run time: 34316 ms
|
TPC-DS: Total hot run time: 194346 ms
|
ClickBench: Total hot run time: 31.09 s
|
run beut |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
run buildall |
TeamCity cloud ut coverage result: |
TPC-H: Total hot run time: 34358 ms
|
TPC-DS: Total hot run time: 185824 ms
|
ClickBench: Total hot run time: 31.52 s
|
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
aws_log_level=2
for logging aws sdk errorS3ObjStorageClient
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)