[GitHub] mikewalch commented on a change in pull request #999: Create 'fluo remove' command #991

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

mikewalch commented on a change in pull request #999: Create 'fluo remove' command #991
URL: https://github.com/apache/fluo/pull/999#discussion_r161252439
 
 

 ##########
 File path: modules/api/src/main/java/org/apache/fluo/api/client/FluoAdmin.java
 ##########
 @@ -119,6 +119,15 @@ public TableExistsException() {
   void initialize(InitializationOptions opts)
       throws AlreadyInitializedException, TableExistsException;
 
+  /**
+   * Removes Fluo application, Accumulo table and shared configuration in Zookeeper. Shared configuration
+   * consists of all properties except those with
+   * {@value org.apache.fluo.api.config.FluoConfiguration#CONNECTION_PREFIX} prefix.
+   * 
+   * @since 1.2.0
+   */
+  void remove();
 
 Review comment:
   I tested the remove command on my machine and everything worked.  An integration test is a great idea but should probably be a separate issue.

----------------------------------------------------------------
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