kuenishi commented on PR #3225: URL: https://github.com/apache/ozone/pull/3225#issuecomment-1333407333
I have force-pushed a rebased version. Please check it out. @sumitagrawl Thank you for the suggestion. I think transaction log index would be enough to guarantee the uniqueness, and all deleted `OmKeyInfo` s in the transaction are included in the `RepeatedOmKeyInfo` object. There are some cases it is unclear which object id should be appended, such as deleting multiple open keys at once in `OMOpenKeysDeleteRequest(WithFSO)`. In such cases, each OmKeyInfos have different object id. -- 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]
