hanishakoneru commented on pull request #3052: URL: https://github.com/apache/ozone/pull/3052#issuecomment-1050049670
> One thing I noticed while running the unit test is even without the changes in TarContainerPacker, the test also passed. Does that mean the key point to this issue is the creation of the CHUNK_PATH in KeyValueContainerUtil? KeyValueContainerUtil change fixes existing containers with missing chunks directory. This could have occurred when a container with empty chunks directory was replicated before. The change in TarContainerPacker is needed so that when containers with empty chunks directories are replicated, the replica container get the chunks directory even though it does not have any contents. -- 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]
