Skip to content

Conversation

@bharatviswa504
Copy link
Contributor

What changes were proposed in this pull request?

Fix calling clean up of few tables which is missing in OzoneManagerDoubleBuffer cleanupcache.

For few tables cleanup of cache is missed:

PrefixTable
S3SecretTable
DelegationTable

What is the link to the Apache JIRA

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

How was this patch tested?

Ran a few integration tests.

Copy link
Contributor

@dineshchitlangia dineshchitlangia left a comment

Choose a reason for hiding this comment

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

+1 LGTM, pending CI

@avijayanhwx
Copy link
Contributor

LGTM +1

Copy link
Contributor

@dineshchitlangia dineshchitlangia left a comment

Choose a reason for hiding this comment

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

New checkstyle issue has been reported regarding incorrect indentation.

Copy link
Contributor

@dineshchitlangia dineshchitlangia left a comment

Choose a reason for hiding this comment

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

+1 checkstyle issues have been addressed.

@dineshchitlangia dineshchitlangia merged commit 6ba7115 into apache:master Nov 1, 2019
ptlrs pushed a commit to ptlrs/ozone that referenced this pull request Mar 8, 2025
…s with size greater than 0 bytes (apache#6447) (apache#112)

Co-authored-by: Sarveksha Yeshavantha Raju <[email protected]>
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.

3 participants