Skip to content

Conversation

@sumitagrawl
Copy link
Contributor

@sumitagrawl sumitagrawl commented Feb 6, 2024

What changes were proposed in this pull request?

for key overwrite case, objectId is same. for deleted key, its suffixed with objectId to be unique. So for continuous create and delete same key, there is a possibility of overwrite deletedTable entry causing missing cleanup of blocks of previous key.

Also snapshot content remain orphan in case of overwrite in active db.

As fix for overwrite, a unique id is appended to key in deleted table to make it unique.

What is the link to the Apache JIRA

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

How was this patch tested?

  • updated existing unit test case

Copy link
Contributor

@sadanand48 sadanand48 left a comment

Choose a reason for hiding this comment

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

Thanks @sumitagrawl for finding & fixing the issue. Overall looks good.

Copy link
Contributor

@sadanand48 sadanand48 left a comment

Choose a reason for hiding this comment

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

LGTM

@sadanand48 sadanand48 merged commit 3d8365f into apache:master Feb 8, 2024
ivandika3 pushed a commit to ivandika3/ozone that referenced this pull request Oct 12, 2025
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