errose28 commented on pull request #3052: URL: https://github.com/apache/ozone/pull/3052#issuecomment-1050064813
Hi @symious following up on your questions: > Does that mean the key point to this issue is the creation of the CHUNK_PATH in KeyValueContainerUtil? There's two parts to this PR: preventing the issue and fixing existing containers. I think [this comment](https://github.com/apache/ozone/pull/3052#discussion_r801921918) might answer the question. > One thing I noticed while running the unit test is even without the changes in TarContainerPacker, the test also passed I cannot repro this. Running master from 68c5ac5df4fbc0edd7114394112fa696a3dc9229 (before this patch was merged), I copy/pasted in the changes to TestKeyValueContainer and both new tests failed. Can you please check again? -- 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]
