sreejasahithi commented on PR #10409: URL: https://github.com/apache/ozone/pull/10409#issuecomment-4628691241
> Thanks @sreejasahithi for the patch. Can you please explain why it is added as `ozone-iceberg` instead of `ozone iceberg`? This duplicates shell script plumbing and we don't get benefits like autocompletion. Thanks for the feedback. I initially added a standalone ozone-iceberg script thinking it would be better to keep outside the main ozone CLI, but I have updated the patch now to use `ozone iceberg rewrite-path` instead, consistent with ozone commands and without duplicating shell plumbing. -- 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]
