Skip to content
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

[AMBARI-25231] : Replace deprecated hadoop commands #2962

Merged
merged 1 commit into from Jun 6, 2019
Merged

[AMBARI-25231] : Replace deprecated hadoop commands #2962

merged 1 commit into from Jun 6, 2019

Conversation

virajjasani
Copy link
Contributor

@virajjasani virajjasani commented May 5, 2019

What changes were proposed in this pull request?

hadoop refreshNodes command is deprecated and it is recommended to start using hdfs command for refreshNodes.

How was this patch tested?

The patch was tested manually. While decommissiong/recommissioning of datanode, refreshNodes command is run on all NameNodes. This was tested with UI. All changes are covered with Unit Tests as well.

Screen Shot 2019-05-05 at 11 31 35 PM

@asfgit
Copy link

asfgit commented May 5, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5189/
Test FAILed.
Test FAILured.

@virajjasani
Copy link
Contributor Author

test is again please

@virajjasani
Copy link
Contributor Author

retest this please

@asfgit
Copy link

asfgit commented May 5, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5192/
Test PASSed.

@virajjasani
Copy link
Contributor Author

Please review @zeroflag @adoroszlai

@zeroflag zeroflag requested review from a user, mpapirkovskyy and zeroflag June 3, 2019 07:21
@zeroflag zeroflag merged commit 1933594 into apache:branch-2.7 Jun 6, 2019
payert pushed a commit to payert/ambari that referenced this pull request Apr 16, 2020
(cherry picked from commit 1933594)

Change-Id: I59a00d25a70d70e356ae06808235549568902cd6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants