Skip to content

HDDS-6998. Ignore keys with higher updateID in OpenKeyDeleteRequest - #3596

Merged
captainzmc merged 2 commits into
apache:masterfrom
kaijchen:HDDS-6998
Jul 14, 2022
Merged

HDDS-6998. Ignore keys with higher updateID in OpenKeyDeleteRequest#3596
captainzmc merged 2 commits into
apache:masterfrom
kaijchen:HDDS-6998

Conversation

@kaijchen

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

Ignore keys with higher updateID in OpenKeyDeleteRequest, see:

https://issues.apache.org/jira/browse/HDDS-6998

How was this patch tested?

TestOMOpenKeysDeleteRequest#testDeleteKeyWithHigherUpdateID()

@kaijchen

Copy link
Copy Markdown
Member Author

Cc @captainzmc PTAL.

@captainzmc
captainzmc self-requested a review July 14, 2022 06:01

@captainzmc captainzmc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1 The change looks good.

@captainzmc

Copy link
Copy Markdown
Member

This bug can cause OM to crash, thanks @kaijchen fix this.

@captainzmc
captainzmc merged commit e1f39bd into apache:master Jul 14, 2022
@kaijchen

Copy link
Copy Markdown
Member Author

Thanks @captainzmc for the review.

@kaijchen
kaijchen deleted the HDDS-6998 branch July 14, 2022 11:15
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.

2 participants