-
Notifications
You must be signed in to change notification settings - Fork 8.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HDDS-1915. Remove hadoop script from ozone distribution #1233
Conversation
🎊 +1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 LGTM.
Ya with this, we can eliminate trunk version of hadoop and hadoop.cmd script.
@arp7 can you also take a look at this change. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
@elek Thanks for the contribution. @bharatviswa504 Thanks for the review. I have committed this to the trunk. |
/bin/hadoop script is included in the ozone distribution even if we a dedicated /bin/ozone
[~arp] reported that it can be confusing, for example "hadoop classpath" returns with a bad classpath (ozone classpath ) should be used instead.
To avoid such confusions I suggest to remove the hadoop script from distribution as ozone script already provides all the functionalities.
It also helps as to reduce the dependencies between hadoop 3.2-SNAPSHOT and ozone as we use the snapshot hadoop script as of now.
See: https://issues.apache.org/jira/browse/HDDS-1915