echonesis commented on PR #8789: URL: https://github.com/apache/ozone/pull/8789#issuecomment-3106003498
> Thanks @echonesis for updating the patch. Just one more question: why does `ContainerChecksums` use `long 0` for unknown data checksum, and `Long null` for unknown/missing metadata checksum? Is there a use case where there is a difference between 0 and `null`? I believe metadata serves a critical role in future operations and analysis. Regarding the `toString()` method, I propose we differentiate between two checksum types: the existing data checksum and a new metadata checksum. I'm treating this as a schema extension, specifically by introducing nullability. This ensures backward compatibility, so existing consumers using older APIs won't experience breakage. -- 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]
