ChenSammi commented on PR #7930:
URL: https://github.com/apache/ozone/pull/7930#issuecomment-2684028294
> * We can check if the path fields are present to make the same
decisions without needing to check a schema version. This is already done in
the current patch in `ContainerDataYaml#readContainerFile`.
@errose28 , it's different, you can check ContainerUtils#verifyChecksum,
where the container yaml file checksum is verified. I would love to adopt it
if it can verify both container yaml file with these two fields and without the
two fields in an elegant way.
--
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]