keith-turner commented on a change in pull request #2225:
URL: https://github.com/apache/accumulo/pull/2225#discussion_r683675526



##########
File path: 
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/Upgrader.java
##########
@@ -30,9 +30,30 @@
  * complete and then be run again later.
  */
 public interface Upgrader {
+
+  /**
+   * Update entries in ZooKeeper - normally runs before the root tablet is 
loaded.
+   *
+   * @param ctx
+   *          the server context.
+   */
   void upgradeZookeeper(ServerContext ctx);
 
+  /**
+   * Update the root tablet - normally runs after the root tablet is loaded 
and before the metadata
+   * table is loaded.
+   *
+   * @param ctx
+   *          the server context.
+   */
   void upgradeRoot(ServerContext ctx);
 
+  /**
+   * Update the metadata table - normally runs after the metadata table is 
loaded and before loading

Review comment:
       ```suggestion
      * Update the metadata table - called after the metadata table is loaded 
and before loading
   ```

##########
File path: 
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/Upgrader.java
##########
@@ -30,9 +30,30 @@
  * complete and then be run again later.
  */
 public interface Upgrader {
+
+  /**
+   * Update entries in ZooKeeper - normally runs before the root tablet is 
loaded.

Review comment:
       ```suggestion
      * Update entries in ZooKeeper - called before the root tablet is loaded.
   ```

##########
File path: 
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/Upgrader.java
##########
@@ -30,9 +30,30 @@
  * complete and then be run again later.
  */
 public interface Upgrader {
+
+  /**
+   * Update entries in ZooKeeper - normally runs before the root tablet is 
loaded.
+   *
+   * @param ctx
+   *          the server context.
+   */
   void upgradeZookeeper(ServerContext ctx);
 
+  /**
+   * Update the root tablet - normally runs after the root tablet is loaded 
and before the metadata

Review comment:
       ```suggestion
      * Update the root tablet - called after the root tablet is loaded and 
before the metadata
   ```




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to