amaliujia commented on pull request #1346:
URL: https://github.com/apache/hadoop-ozone/pull/1346#issuecomment-700441836
Re @bshashikant
Agreed. Right now the command itself is unified (for both OM and SCM, we
name this command as `roles`). Then we should unify the behavior fo both
amaliujia commented on pull request #1346:
URL: https://github.com/apache/hadoop-ozone/pull/1346#issuecomment-699688323
@timmylicheng comments addressed. Can you take another look?
This is an automated message from the
amaliujia commented on pull request #1346:
URL: https://github.com/apache/hadoop-ozone/pull/1346#issuecomment-689826469
R @timmylicheng @nandakumar131
I am thinking maybe we can first merge this PR and create a JIRA to track
left work. Right now per feedback this command could
amaliujia commented on pull request #1346:
URL: https://github.com/apache/hadoop-ozone/pull/1346#issuecomment-689140982
Also R: @nandakumar131 can you please take a look?
This is an automated message from the Apache Git
amaliujia commented on pull request #1346:
URL: https://github.com/apache/hadoop-ozone/pull/1346#issuecomment-685425652
Uploaded one commit to
1. use `ozone admin scm roles`
2. fix the acceptance test.
This is an
amaliujia commented on pull request #1346:
URL: https://github.com/apache/hadoop-ozone/pull/1346#issuecomment-685173208
Addressed the following comments
1. Merge `getRatisStatus` with `GetScmInfo`
2. Adopt command syntax `ozone admin scm status`
3. added an acceptance test