Yikun commented on pull request #32604: URL: https://github.com/apache/spark/pull/32604#issuecomment-845185892
> Yes the second idea is separate - just wondering if you had a second to add that during this change. I don't mind warning if it's not found, and proceeding. If it's not found, the build mvn would be breaked with exit 2. Then, the developer can install the shasum mannually. So, command -v is not very important in here, I prefer to keep current change in here. ( Sorry for typing slowly in mobile phone, and if any other followup, I will address after 12 hours when I back to my dev PC, or you can feel free to help to change this pr. : ) ) -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
