DomGarguilo commented on a change in pull request #2426:
URL: https://github.com/apache/accumulo/pull/2426#discussion_r789943249



##########
File path: 
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/Upgrader9to10.java
##########
@@ -111,6 +111,31 @@
  * {@link #dropSortedMapWALFiles(ServerContext)}. For more information see the 
following issues:
  * <a href="https://github.com/apache/accumulo/issues/2117";>#2117</a> and
  * <a href="https://github.com/apache/accumulo/issues/2179";>#2179</a>
+ *
+ * The following methods were created for External compactions:
+ * {@link #createExternalCompactionNodes(ServerContext)}, {@link 
#setMetaTableProps(ServerContext)}
+ *
+ * Improvements to the metadata and root tables were made in this version. See 
pull request
+ * <a href="https://github.com/apache/accumulo/pull/1174";>#1174</a> for more 
details. The
+ * {@link #upgradeRootTabletMetadata(ServerContext)} method was added for this 
change. This change
+ * allowed other changes to how volumes were stored in the metadata and have 
Accumulo always call
+ * the volume chooser for new tablet files. This change was facilitated with 
the
+ * {@link #upgradeDirColumns(ServerContext, Ample.DataLevel)} method and done 
in
+ * <a href="https://github.com/apache/accumulo/pull/1389";>#1389</a>
+ *
+ * The method {@link #upgradeRelativePaths(ServerContext, Ample.DataLevel)} 
was added for resolving
+ * and replacing all relative tablet file paths found in metadata tables with 
absolute paths during
+ * upgrade. Absolute paths are resolved by prefixing relative paths with a 
volume configured by the
+ * user in a the instance.volumes.upgrade.relative property, which is only 
used during an upgrade.

Review comment:
       ```suggestion
    * user in the instance.volumes.upgrade.relative property, which is only 
used during an upgrade.
   ```




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