(accumulo) branch 2.1 updated: Changes exception handling for deleted table log message (#6580)

[ddanielr]

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

ddanielr 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 c1c6ddc3eb Changes exception handling for deleted table log message 
(#6580)
c1c6ddc3eb is described below

commit c1c6ddc3eb49eef8288cad673f85d66096a30b52
Author: Daniel Roberts <ddani...@gmail.com>
AuthorDate: Fri Oct 9 14:51:52 2026 -0400

    Changes exception handling for deleted table log message (#6580)
    
    * Changed the error handling to handle NullPointerException instead
---
 .../tserver/memory/LargestFirstMemoryManager.java       | 17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)

diff --git 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/memory/LargestFirstMemoryManager.java
 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/memory/LargestFirstMemoryManager.java
index 40a85110fe..44ce6942e7 100644
--- 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/memory/LargestFirstMemoryManager.java
+++ 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/memory/LargestFirstMemoryManager.java
@@ -24,7 +24,6 @@ import java.util.List;
 import java.util.Map.Entry;
 import java.util.TreeMap;
 
-import org.apache.accumulo.core.client.TableNotFoundException;
 import org.apache.accumulo.core.conf.Property;
 import org.apache.accumulo.core.data.TableId;
 import org.apache.accumulo.core.dataImpl.KeyExtent;
@@ -194,17 +193,11 @@ public class LargestFirstMemoryManager {
           if (idleTime > getMinCIdleThreshold(tablet)) {
             largestIdleMemTablets.put(timeMemoryLoad, tabletInfo);
           }
-        } catch (IllegalArgumentException e) {
-          Throwable cause = e.getCause();
-          if (cause != null && cause instanceof TableNotFoundException) {
-            log.trace("Ignoring extent for deleted table: {}", tablet);
-
-            // The table might have been deleted during the iteration of the 
tablets
-            // We just want to eat this exception, do nothing with this 
tablet, and continue
-            continue;
-          }
-
-          throw e;
+        } catch (NullPointerException e) {
+          log.trace("Table config is null. Ignoring extent for deleted table: 
{}", tablet);
+          // The table might have been deleted during the iteration of the 
tablets
+          // We just want to eat this exception, do nothing with this tablet, 
and continue
+          continue;
         }
         // Only place the tablet into largestMemTablets map when the table 
still exists
         largestMemTablets.put(timeMemoryLoad, tabletInfo);