k5342 opened a new pull request, #4530: URL: https://github.com/apache/ozone/pull/4530
## What changes were proposed in this pull request? This is a workaround for HDDS-8371 when we have a dirty OMKeyInfo entry in the keyTable. The dirty in this context means keyName includes a prefix to the key in the keyName. As the listStatus API assumes non-prefixed keys to concatenate keys, we eventually get a wrong result from listStatus. As a workaround, we can use fileName instead of keyName. I noticed keyInfo#fileName is not dirty in this case, which means does not include the prefix. I think this is a short-term fix for this problem. But I'm not confident about how this change is valuable because it is something ad-hoc. The background is described as the above and Jira, so could you give me some comments? For the root cause, I don't make sure whether this is from persistent information or dynamically generated information, but it seems OMKeyInfo#fromPersistedFormat() returns the dirty result. We need to fix the root cause later by reviewing the write path for keys. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-8371 Please replace this section with the link to the Apache JIRA) ## How was this patch tested? - `mvn test` locally - Unit Test triggered by CI -- 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]
