gszadovszky commented on code in PR #3101: URL: https://github.com/apache/parquet-java/pull/3101#discussion_r1882247074
########## dev/README.md: ########## @@ -91,3 +91,61 @@ Merge hash: 485658a5 Would you like to pick 485658a5 into another branch? (y/n): ``` For now just say n as we have 1 branch + +# Release Verification Review Comment: I agree the "release verification" should be moved to the `parquet-site` repo instead. Then, we even can have a link to this section in the VOTE email template. I'm not sure why we need to check for the license headers separately in the tarball. It is already in the build process so we shall not have license header issues in the repo. What I usually do instead is comparing the content of the tarball with a freshly cloned repo set to the release RC tag. There should be no differences. -- 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]
