-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Segment Replication] Refactor RemoteStoreReplicationSource #8767
Conversation
Gradle Check (Jenkins) Run Completed with:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall PR looks much better. Have few minor comments.
server/src/main/java/org/opensearch/indices/replication/RemoteStoreReplicationSource.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/opensearch/indices/replication/RemoteStoreReplicationSource.java
Outdated
Show resolved
Hide resolved
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Signed-off-by: Suraj Singh <[email protected]>
Signed-off-by: Suraj Singh <[email protected]>
Signed-off-by: Suraj Singh <[email protected]>
Signed-off-by: Suraj Singh <[email protected]>
Signed-off-by: Suraj Singh <[email protected]>
Signed-off-by: Suraj Singh <[email protected]>
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Signed-off-by: Suraj Singh <[email protected]>
Gradle Check (Jenkins) Run Completed with:
|
|
@gaiksaya: I see you added this check via #8486. Can you confirm why this task is failing ?
|
Gradle Check (Jenkins) Run Completed with:
|
Compatibility status:
|
Gradle Check (Jenkins) Run Completed with:
|
The backport to
To backport manually, run these commands in your terminal: # Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/backport-2.x 2.x
# Navigate to the new working tree
pushd ../.worktrees/backport-2.x
# Create a new branch
git switch --create backport/backport-8767-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 57eb105a6a156e3a313efd58e267d772ac178eea
# Push it to GitHub
git push --set-upstream origin backport/backport-8767-to-2.x
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/backport-2.x Then, create a pull request where the |
…ch-project#8767) * [Segment Replication] Refactor remote replication source Signed-off-by: Suraj Singh <[email protected]> * Unit test updates Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Segregate shard level tests for node to node and remote store segment replication Signed-off-by: Suraj Singh <[email protected]> * Fix failing unit tests Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Fix more unit tests Signed-off-by: Suraj Singh <[email protected]> * Improve RemoteStoreReplicationSourceTests, remove unnecessary mocks and use actual failures for failure/exception use cases Signed-off-by: Suraj Singh <[email protected]> * Spotless check fix Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Ignore files already in store while computing segment file diff with primary Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Move read/writes from IndexInput/Output to RemoteSegmentMetadata Signed-off-by: Suraj Singh <[email protected]> * Address review commnt Signed-off-by: Suraj Singh <[email protected]> * Update recovery flow to perform commits during recovery Signed-off-by: Suraj Singh <[email protected]> * Remove un-necessary char Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Update comment nit-pick Signed-off-by: Suraj Singh <[email protected]> * Remove deletion logic causing read issues due to deleted segments_N Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix unit tests Signed-off-by: Suraj Singh <[email protected]> --------- Signed-off-by: Suraj Singh <[email protected]>
…ch-project#8767) * [Segment Replication] Refactor remote replication source Signed-off-by: Suraj Singh <[email protected]> * Unit test updates Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Segregate shard level tests for node to node and remote store segment replication Signed-off-by: Suraj Singh <[email protected]> * Fix failing unit tests Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Fix more unit tests Signed-off-by: Suraj Singh <[email protected]> * Improve RemoteStoreReplicationSourceTests, remove unnecessary mocks and use actual failures for failure/exception use cases Signed-off-by: Suraj Singh <[email protected]> * Spotless check fix Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Ignore files already in store while computing segment file diff with primary Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Move read/writes from IndexInput/Output to RemoteSegmentMetadata Signed-off-by: Suraj Singh <[email protected]> * Address review commnt Signed-off-by: Suraj Singh <[email protected]> * Update recovery flow to perform commits during recovery Signed-off-by: Suraj Singh <[email protected]> * Remove un-necessary char Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Update comment nit-pick Signed-off-by: Suraj Singh <[email protected]> * Remove deletion logic causing read issues due to deleted segments_N Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix unit tests Signed-off-by: Suraj Singh <[email protected]> --------- Signed-off-by: Suraj Singh <[email protected]>
…ch-project#8767) * [Segment Replication] Refactor remote replication source Signed-off-by: Suraj Singh <[email protected]> * Unit test updates Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Segregate shard level tests for node to node and remote store segment replication Signed-off-by: Suraj Singh <[email protected]> * Fix failing unit tests Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Fix more unit tests Signed-off-by: Suraj Singh <[email protected]> * Improve RemoteStoreReplicationSourceTests, remove unnecessary mocks and use actual failures for failure/exception use cases Signed-off-by: Suraj Singh <[email protected]> * Spotless check fix Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Ignore files already in store while computing segment file diff with primary Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Move read/writes from IndexInput/Output to RemoteSegmentMetadata Signed-off-by: Suraj Singh <[email protected]> * Address review commnt Signed-off-by: Suraj Singh <[email protected]> * Update recovery flow to perform commits during recovery Signed-off-by: Suraj Singh <[email protected]> * Remove un-necessary char Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Update comment nit-pick Signed-off-by: Suraj Singh <[email protected]> * Remove deletion logic causing read issues due to deleted segments_N Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix unit tests Signed-off-by: Suraj Singh <[email protected]> --------- Signed-off-by: Suraj Singh <[email protected]>
…9076) * [Segment Replication] Refactor RemoteStoreReplicationSource (#8767) * [Segment Replication] Refactor remote replication source Signed-off-by: Suraj Singh <[email protected]> * Unit test updates Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Segregate shard level tests for node to node and remote store segment replication Signed-off-by: Suraj Singh <[email protected]> * Fix failing unit tests Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Fix more unit tests Signed-off-by: Suraj Singh <[email protected]> * Improve RemoteStoreReplicationSourceTests, remove unnecessary mocks and use actual failures for failure/exception use cases Signed-off-by: Suraj Singh <[email protected]> * Spotless check fix Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Ignore files already in store while computing segment file diff with primary Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Move read/writes from IndexInput/Output to RemoteSegmentMetadata Signed-off-by: Suraj Singh <[email protected]> * Address review commnt Signed-off-by: Suraj Singh <[email protected]> * Update recovery flow to perform commits during recovery Signed-off-by: Suraj Singh <[email protected]> * Remove un-necessary char Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Update comment nit-pick Signed-off-by: Suraj Singh <[email protected]> * Remove deletion logic causing read issues due to deleted segments_N Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix unit tests Signed-off-by: Suraj Singh <[email protected]> --------- Signed-off-by: Suraj Singh <[email protected]> * Rebase 2.x and resolve conflicts Signed-off-by: Suraj Singh <[email protected]> * Fix unit test Signed-off-by: Suraj Singh <[email protected]> --------- Signed-off-by: Suraj Singh <[email protected]>
…ch-project#8767) * [Segment Replication] Refactor remote replication source Signed-off-by: Suraj Singh <[email protected]> * Unit test updates Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Segregate shard level tests for node to node and remote store segment replication Signed-off-by: Suraj Singh <[email protected]> * Fix failing unit tests Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Fix more unit tests Signed-off-by: Suraj Singh <[email protected]> * Improve RemoteStoreReplicationSourceTests, remove unnecessary mocks and use actual failures for failure/exception use cases Signed-off-by: Suraj Singh <[email protected]> * Spotless check fix Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Ignore files already in store while computing segment file diff with primary Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Move read/writes from IndexInput/Output to RemoteSegmentMetadata Signed-off-by: Suraj Singh <[email protected]> * Address review commnt Signed-off-by: Suraj Singh <[email protected]> * Update recovery flow to perform commits during recovery Signed-off-by: Suraj Singh <[email protected]> * Remove un-necessary char Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Update comment nit-pick Signed-off-by: Suraj Singh <[email protected]> * Remove deletion logic causing read issues due to deleted segments_N Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix unit tests Signed-off-by: Suraj Singh <[email protected]> --------- Signed-off-by: Suraj Singh <[email protected]> Signed-off-by: Kaushal Kumar <[email protected]>
…ch-project#8767) * [Segment Replication] Refactor remote replication source Signed-off-by: Suraj Singh <[email protected]> * Unit test updates Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Segregate shard level tests for node to node and remote store segment replication Signed-off-by: Suraj Singh <[email protected]> * Fix failing unit tests Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Fix more unit tests Signed-off-by: Suraj Singh <[email protected]> * Improve RemoteStoreReplicationSourceTests, remove unnecessary mocks and use actual failures for failure/exception use cases Signed-off-by: Suraj Singh <[email protected]> * Spotless check fix Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Ignore files already in store while computing segment file diff with primary Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Move read/writes from IndexInput/Output to RemoteSegmentMetadata Signed-off-by: Suraj Singh <[email protected]> * Address review commnt Signed-off-by: Suraj Singh <[email protected]> * Update recovery flow to perform commits during recovery Signed-off-by: Suraj Singh <[email protected]> * Remove un-necessary char Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Update comment nit-pick Signed-off-by: Suraj Singh <[email protected]> * Remove deletion logic causing read issues due to deleted segments_N Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix unit tests Signed-off-by: Suraj Singh <[email protected]> --------- Signed-off-by: Suraj Singh <[email protected]> Signed-off-by: Ivan Brusic <[email protected]>
…ch-project#8767) * [Segment Replication] Refactor remote replication source Signed-off-by: Suraj Singh <[email protected]> * Unit test updates Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Self review Signed-off-by: Suraj Singh <[email protected]> * Segregate shard level tests for node to node and remote store segment replication Signed-off-by: Suraj Singh <[email protected]> * Fix failing unit tests Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Fix more unit tests Signed-off-by: Suraj Singh <[email protected]> * Improve RemoteStoreReplicationSourceTests, remove unnecessary mocks and use actual failures for failure/exception use cases Signed-off-by: Suraj Singh <[email protected]> * Spotless check fix Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Ignore files already in store while computing segment file diff with primary Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix failing UT Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Move read/writes from IndexInput/Output to RemoteSegmentMetadata Signed-off-by: Suraj Singh <[email protected]> * Address review commnt Signed-off-by: Suraj Singh <[email protected]> * Update recovery flow to perform commits during recovery Signed-off-by: Suraj Singh <[email protected]> * Remove un-necessary char Signed-off-by: Suraj Singh <[email protected]> * Address review comments Signed-off-by: Suraj Singh <[email protected]> * Update comment nit-pick Signed-off-by: Suraj Singh <[email protected]> * Remove deletion logic causing read issues due to deleted segments_N Signed-off-by: Suraj Singh <[email protected]> * Spotless fix Signed-off-by: Suraj Singh <[email protected]> * Fix unit tests Signed-off-by: Suraj Singh <[email protected]> --------- Signed-off-by: Suraj Singh <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
Description
Refactors the RemoteStoreReplicationSource to align with existing node node implementation.
RemoteSegmentMetadata
to store ReplicationCheckpointRequest review from
Related Issues
Resolves #7802 #8452
Check List
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.