smengcl commented on PR #4266: URL: https://github.com/apache/ozone/pull/4266#issuecomment-1447028537
> > Got it. I was not aware of this new usage of RepeatedOmKeyInfo before this. In this case I agree that it should be properly handled like in this PR. > > Just want to be sure I understand. Does that mean that the value in the KeyTable here: > > https://github.com/apache/ozone/blob/c755e5d6459af8abc0ba634d7901a497a80d0f72/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/codec/OMDBDefinition.java#L89-L96 > > is changing from OmKeyInfo to RepeatedOmKeyInfo? No. None of the table schema has changed. The deletion in OmKeyCommit is done to deletedTable. However, that still doesn't explain why the table filter didn't work. I still [suspect](https://github.com/apache/ozone/pull/4266#discussion_r1107680066) the event source is the culprit. If have a repro it will be easy to dig into that. -- 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]
