Refactor the Server Command-Lines for Config-Node, Data-Node and the upcoming AI-Node
Open
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 6.4k
- Forks
- 1.2k
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 115
Description
Currently the server CLI implementation of the existing nodes lacks clarity and needs a major cleaning up.
- Use Apache Commons CLI to implement the general command-line functionality
- Refactor the ConfigNode, DataNode and AiNode CLI parts
The syntax of the commandline issues will change to the follwing format:
{NodeCli} <options> <command>
The commands will be:
- start
- remove
The options will be:
- for the remove command
- nodeId {some node id}
To be discussed:
- Should it be possible to remove multiple nodes, by passing in a list of ids?
- Should it be possible to remove the current node, by omitting the nodeId parameter?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the existing ConfigNode, DataNode, and AiNode server CLI implementations and compare their current command-line behavior. Use the issue's proposed start/remove command structure and nodeId option as the target, while resolving whether multiple-node removal and omission of nodeId are supported; done means the three CLIs share the clarified Apache Commons CLI-based syntax.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- cli
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100