store: quickly fetch needed file on S3 through Marker#897
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #897 +/- ##
=======================================
Coverage ? 67.37%
=======================================
Files ? 135
Lines ? 13100
Branches ? 0
=======================================
Hits ? 8826
Misses ? 3663
Partials ? 611
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Yang Keao <yangkeao@chunibyo.icu>
|
/hold There is a little timezone issue. I realized it while dreaming 😂. Let me fix it in the morning. |
|
/unhold No issue, bad dream 😠 . |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: djshow832 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[LGTM Timeline notifier]Timeline:
|
What problem does this PR solve?
Issue Number: close #892
Problem Summary:
What is changed and how it works:
This PR is based on the assumption that:
The file name format is similar to RFC3339 (ISO8601), and RFC3339 (ISO8601)'s order is compatible with lexicographical order if there is no timezone / time shift.
Check List
Tests
docker run -p 9091:9090 -p 9191:9191 -t adobe/s3mock:4.8.0.aws s3api create-bucket --bucket my-bucket --endpoint-url=http://localhost:9091for f in *;do curl --request PUT --upload-file ./$f http://localhost:9091/my-bucket/$f; done./replayer --command-start-time 2025-09-15T22:06:10.044+08:00 --format audit_log_plugin --input s3://my-bucket?endpoint=http://localhost:9091\®ion=us-east-1\&access-key=a\&secret-access-key=b\&force-path-style=1Notable changes
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.