k5342 commented on PR #4723: URL: https://github.com/apache/ozone/pull/4723#issuecomment-1551725820
@adoroszlai Thank you for your pointer. Sounds good to me for keeping a null than remove a null. My initial thought is to avoid `null` by overwriting it at the `setSchemaVersion` could fix the overall NPE related to the schema field, but actually it requires rewriting checksum computation, and the change was bigger than I expected. -- 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]
