rahil-c commented on code in PR #7224: URL: https://github.com/apache/hudi/pull/7224#discussion_r1029523561
########## packaging/hudi-cli-bundle/hudi-cli-bundle.sh: ########## @@ -0,0 +1,30 @@ +#!/usr/bin/env bash Review Comment: Im ok with either, I think I just wanted the name to be different from the `hudi-cli.sh` that is located in the `hudi-cli` module so that people do not get confused. I think maintaining two scripts with the same name might be confusing and im wondering if we should name it specific to the platform hudi is running on? Something like `hudi-cli-emr.sh`? -- 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]
