(accumulo) branch 2.1 updated: Remove unnecessary read of time column during delete (#4008)

[cshannon]

This is an automated email from the ASF dual-hosted git repository.

cshannon pushed a commit to branch 2.1
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/2.1 by this push:
     new f8cb312032 Remove unnecessary read of time column during delete (#4008)
f8cb312032 is described below

commit f8cb312032d413f00f517f73f662edb0bf16c16c
Author: Christopher L. Shannon (cshannon) <christopher.l.shan...@gmail.com>
AuthorDate: Sat Dec 2 09:49:20 2023 -0500

    Remove unnecessary read of time column during delete (#4008)
---
 .../main/java/org/apache/accumulo/manager/TabletGroupWatcher.java    | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git 
a/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
 
b/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
index 3295cb9604..81744441aa 100644
--- 
a/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
+++ 
b/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
@@ -673,7 +673,6 @@ abstract class TabletGroupWatcher extends 
AccumuloDaemonThread {
     KeyExtent extent = info.getExtent();
     String targetSystemTable = extent.isMeta() ? RootTable.NAME : 
MetadataTable.NAME;
     Manager.log.debug("Deleting tablets for {}", extent);
-    MetadataTime metadataTime = null;
     KeyExtent followingTablet = null;
     if (extent.endRow() != null) {
       Key nextExtent = new Key(extent.endRow()).followingKey(PartialKey.ROW);
@@ -710,8 +709,6 @@ abstract class TabletGroupWatcher extends 
AccumuloDaemonThread {
             ample.putGcFileAndDirCandidates(extent.tableId(), 
datafilesAndDirs);
             datafilesAndDirs.clear();
           }
-        } else if (ServerColumnFamily.TIME_COLUMN.hasColumns(key)) {
-          metadataTime = MetadataTime.parse(entry.getValue().toString());
         } else if (isTabletAssigned(key)) {
           throw new IllegalStateException(
               "Tablet " + key.getRow() + " is assigned during a merge!");
@@ -740,7 +737,7 @@ abstract class TabletGroupWatcher extends 
AccumuloDaemonThread {
 
       // If there is another tablet after the delete range then update the 
prev end row
       // of that tablet as all tablets will have been deleted in the delete 
range.
-      // If the delete range includes the last tablet in the tablet then we 
need
+      // If the delete range includes the last tablet in the table then we need
       // to update the last tablet's previous end row as deleteTablets() will 
keep the
       // last tablet and only delete the files as we require at least 1 tablet 
to exist
       final KeyExtent goalTablet;