Skip to content

Remove duplicate wait time on translog archive batch#2

Open
axieatlas wants to merge 2 commits into
feature/remote-store-zip-upload-2.17from
axie/remote-store-zip
Open

Remove duplicate wait time on translog archive batch#2
axieatlas wants to merge 2 commits into
feature/remote-store-zip-upload-2.17from
axie/remote-store-zip

Conversation

@axieatlas

Copy link
Copy Markdown

Description

The buffered interval time has been waited on BufferedAsyncIOProcessorBatching, translog archive batch coordiantor does not need to wait for another buffer interval to batch multiple shards translogs.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

The buffered interval time has been waited on BufferedAsyncIOProcessorBatching, translog archive batch coordiantor does not need to wait for another buffer interval to batch multiple shards translogs.
… batch is uploaded will be uploaded immediately after the first batch completes

No caller of timerDispatcher in code
hnguyen10atlassian added a commit that referenced this pull request May 27, 2026
…reshold

Wire fallbackStrategies() in downloadOnce():
  - If primary strategy's downloadRange() throws IOException, core iterates
    strategy.fallbackStrategies() in order until one succeeds.
  - Enables safe rolling upgrades: TAR strategy can declare DefaultRemote
    StoreTranslogStrategy as fallback so per-file blobs written before the
    strategy switch are still recoverable.
  - Default fallbackStrategies() returns empty list — no change for stable
    strategy deployments.

Note: globalCheckpoint wiring into ShardReport (Fix #2) requires threading
through TransferSnapshot → TranslogTransferMetadata which touches many
construction sites. Deferred to a follow-up — current default of Long.MAX_VALUE
is conservative (never GCs early) so correctness is maintained.
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.

1 participant