HeartSaVioR commented on pull request #32401: URL: https://github.com/apache/spark/pull/32401#issuecomment-882199840
I haven't looked into the code deeply (I just helped to fix RAT issue) but IMHO the value of this PR (worth to ship 3.2 or not) depends on whether we "verify" the checksum or not. If we only write the checksum and not yet verify it, nothing is changed yet in end users' point of view and we should wait for next PR(s) to be completed. If this PR introduces the checksum verification as well (and proper error message), personally this PR itself seems to worth to ship without waiting for other PRs. -- 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]
