Russole opened a new pull request, #9291: URL: https://github.com/apache/ozone/pull/9291
## What changes were proposed in this pull request? This PR continues the effort from [HDDS-10519](https://issues.apache.org/jira/browse/HDDS-10519) to make OmBucketInfo fully immutable and to eliminate in-place mutations that can lead to subtle state inconsistencies. * Change OMBucketSetOwnerRequest and OMKeyRenameRequestWithFSO to not modify OmBucketInfo in place. Instead, convert the object to a builder via OmBucketInfo.toBuilder(), set the updated properties on the builder, then builld new OmBucketInfo instance. perspective not just for the reviewer. * Remove setModificationTime and setOwner from OmBucketInfo. * Change owner and modificationTime in OmBucketInfo to be final. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-13916 ## How was this patch tested? This patch is a non-functional cleanup. I relied on existing unit and integration tests, and verified that the full CI passed on my fork. Please let me know if additional tests are needed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
