Skip to content

Verify remote translog size-based flushing - #3

Closed
shourya035 wants to merge 1 commit into
remote-translog-byte-tracking-flowfrom
remote-translog-byte-tracking-it
Closed

Verify remote translog size-based flushing#3
shourya035 wants to merge 1 commit into
remote-translog-byte-tracking-flowfrom
remote-translog-byte-tracking-it

Conversation

@shourya035

@shourya035 shourya035 commented Aug 21, 2026

Copy link
Copy Markdown
Owner

Description

Adds remote-store integration coverage for both sides of the opt-in behavior.

  • Verifies the default-disabled path preserves the legacy moving-retention-boundary behavior without a periodic flush.
  • Verifies enabled byte tracking accumulates bytes across refreshes and triggers a periodic flush.
  • Verifies a successful commit clears the size trigger.
  • Documents each integration scenario with point-by-point Javadocs.

Related issue

Tracks opensearch-project#22802.

Stack

  1. Track translog bytes across commits聽#1 Byte tracking abstraction
  2. Restore size-based remote translog flushes聽#2 Remote translog flow and opt-in setting
  3. Verify remote translog size-based flushing聽#3 Remote-store integration coverage (this PR)

Depends on #2.

Testing

  • ./gradlew :server:internalClusterTest --tests "org.opensearch.remotestore.RemoteStoreIT.testRemoteTranslogSizeFlushUsesLegacyRetentionBoundaryWhenDisabled" --tests "org.opensearch.remotestore.RemoteStoreIT.testRemoteTranslogSizeFlushTracksBytesAcrossRefreshes"
  • ./gradlew :test:framework:precommit

Checklist

  • Functionality includes integration tests.
  • Commits are signed off.

Cover legacy moving-boundary behavior with byte tracking disabled.

Verify accumulated bytes trigger a commit when tracking is enabled.

Signed-off-by: Shourya Dutta Biswas <114977491+shourya035@users.noreply.github.com>
@shourya035 shourya035 closed this Aug 21, 2026
@shourya035
shourya035 deleted the remote-translog-byte-tracking-it branch August 21, 2026 10:46
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