apache / apache/accumulo

Remove use of SiteConfiguration in shell's FateCommand

Open
#2,155 0 comments 0 reactions 1 assignee Claimed by @Manno15 View on GitHub
bug
Dominant language
Java
Stars
1.2k
Forks
487
Avg merge
4d 5h
Merged PRs (30d)
13

Description

SiteConfiguration is a configuration type intended for server-side code. It is not expected that client-side code will have access to the server-side configuration file. Therefore, it doesn't make sense that the shell, which is a client-side application, should leverage the code in the SiteConfiguration class.

From what I can tell, this is the only remaining non-deprecated use of SiteConfiguration in our client code.

It should be removed.

However, it may not be easy, since the implementation of the command relies on the ability to write directly to ZooKeeper in an area that is intended to be used by server-side utilities. In order to make this command purely client-side, it may need to be written to use RPC calls to a server, so that it does not presume access to the server-side configuration file, handled by the SiteConfiguration class.

(was: https://issues.apache.org/jira/browse/ACCUMULO-2996)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.