apache/accumulo

Service status command is not functional in elasticity after merge

keith-turner opened this issue · 2 comments

Describe the bug

In commit c54d544 the changes from #4567 were merged into elasticity. The changes did not compile because the compaction coordinator no longer exists as separate process in elasticity. Rather than attempt to fix this in a merge commit, changes were made to get the code compiling that likely left the command non functional. Probably need to remove a lot of code and test to properly fix this.

The following are the two changes that made things compile, but left things in a broken state.

StatusSummary getCoordinatorStatus(final ZooReader zooReader, String zRootPath) {
throw new UnsupportedOperationException();
}

// String lockPath = zRoot + Constants.ZCOORDINATOR_LOCK;
String lockPath = null;

Expected behavior
The service status command works in elasticity.

On it.

This requires 814ac68 to be merged first which should be most of the changes becuase of ServiceLock usage changes. After that is merged, the removal of the coordinator should just be some deletes - the methods that collects the info getCoordinatorStatus, places where it is called and removal of the test case.