[GitHub] sijie opened a new pull request #1093: BP-27: New BookKeeper CLI

[GitBox <gi...@apache.org>]

sijie opened a new pull request #1093: BP-27: New BookKeeper CLI
URL: https://github.com/apache/bookkeeper/pull/1093
 
 
   Descriptions of the changes in this PR:
   
   `BookieShell` is the current bookkeeper cli for interacting and operating a bookkeeper cluster. However, this class is getting bigger with more commands added to it. It is facing a few problems for maintenance and extensibility.
   
   - All commands sit in one gaint shell class. It is hard to tell if a command is used for managing a bookie only or if a command is used for managing a cluster.
   - Lack of unit tests. This class has very few test coverage. Most of the commands (introduced in early days) don't have a unit test.
   - Lack of extensibility. If a new function component (for example, dlog) is introduced, it is a bit hard to extend this CLI to have commands for new function component.
   
   All these problems lead to the proposal here. This proposal is to propose refactoring/redesigning the bookkeeper CLI to allow better managebility for maintenance, better test coverage and better extensibility for new function components.
   
   Master Issue: #1000 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services