more cleanup

keith-turner · Sep 30, 2024 · ec0febb · ec0febb
1 parent 4285753
commit ec0febb
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/core/src/main/java/org/apache/accumulo/core/client/rfile/LoadPlanCollector.java b/core/src/main/java/org/apache/accumulo/core/client/rfile/LoadPlanCollector.java
@@ -71,7 +71,6 @@ private void appendSplits(Key key) {
       var extent = new KeyExtent(FAKE_ID, tableSplits.getEndRow(), tableSplits.getPrevRow());
       Preconditions.checkState(extent.contains(row), "%s does not contain %s", tableSplits, row);
       if (currentExtent != null) {
-        // TODO validate that row is after the currentExtent
         overlappingExtents.add(currentExtent);
       }
       currentExtent = extent;

diff --git a/core/src/main/java/org/apache/accumulo/core/client/rfile/RFileWriter.java b/core/src/main/java/org/apache/accumulo/core/client/rfile/RFileWriter.java
@@ -272,9 +272,8 @@ public void close() throws IOException {
    * @param filename
    * @return load plan computed from the keys written to the rfile.
    * @see org.apache.accumulo.core.client.rfile.RFile.WriterOptions#withSplitResolver(LoadPlan.SplitResolver)
-   *      TODO since tags on all new apis
+   * @since 2.1.4
    */
-  // TODO test case of empty rfile... and just test this in general
   public LoadPlan getLoadPlan(String filename) {
     return loadPlanCollector.getLoadPlan(filename);
   }