Skip to content
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

[Remove] Deprecated Synced Flush API #1761

Merged
merged 3 commits into from
Jan 27, 2022

Conversation

nknize
Copy link
Collaborator

@nknize nknize commented Dec 17, 2021

This PR removes the deprecated synced flush API which was replaced by sequence number and retention lease mechanisms and no longer used in 2.0.0.

NOTE: this is not a blocker for Lucene 9 or 6.x deprecation removal

relates: #1674

@nknize nknize added v2.0.0 Version 2.0.0 non-issue bugs / unexpected behaviors that end up non issues; audit trail simple changes that aren't issues labels Dec 17, 2021
@nknize nknize requested a review from a team as a code owner December 17, 2021 17:43
@opensearch-ci-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@nknize nknize changed the title [Remove] Deprecated Sync Flush API [Remove] Deprecated Synced Flush API Dec 17, 2021
@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 57d27afd9aa7d5db8ef400e56e3751ae0efaaed6
Log 1573

Reports 1573

@nknize nknize marked this pull request as draft December 17, 2021 18:18
@andrross
Copy link
Member

I went looking for any documentation for this API, and the only reference I found was in the security plugin. Will we need a follow up task to clean that up?

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 19a7d655fd8e01778dd39113fbbbcf5d5de230c6
Log 1915

Reports 1915

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 1b6cbfbe46ce6a594245c48ff09fe974af930ae2
Log 1930

Reports 1930

This commit removes the deprecated sync flush API which was replaced by sequence
number and retention lease mechanisms and no longer used in 2.0.0.

Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
@nknize nknize marked this pull request as ready for review January 18, 2022 19:09
@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success c0b6125
Log 1968

Reports 1968

@nknize
Copy link
Collaborator Author

nknize commented Jan 18, 2022

I went looking for any documentation for this API, and the only reference I found was in the security plugin. Will we need a follow up task to clean that up?

Looks like it's just an access permission for the _flush/synced endpoint? I think the security plugin should retain this so the endpoint can still be accessed and users receive the helpful endpoint removal warning (which we can fully remove in 3.0.0):

Synced flush was removed and a normal flush was performed instead. This transition will be removed in a future version.

@nknize
Copy link
Collaborator Author

nknize commented Jan 25, 2022

This is ready for review.

Copy link
Contributor

@adnapibar adnapibar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backwards-compatibility non-issue bugs / unexpected behaviors that end up non issues; audit trail simple changes that aren't issues v2.0.0 Version 2.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants