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

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


The following commit(s) were added to refs/heads/main by this push:
     new 2d7a63bf07 removes vestigial cache clear in bulk import (#3339)
2d7a63bf07 is described below

commit 2d7a63bf07ab99ce00609e18203d78ddeb9aa092
Author: Keith Turner <ktur...@apache.org>
AuthorDate: Tue Apr 25 14:41:40 2023 -0400

    removes vestigial cache clear in bulk import (#3339)
    
    While working on #3337 code that cleared the table id cache for no
    apparent reason was found.  Investigation indicates that the cache clear
    used to happen before a cache read. The cache read was removed, but the
    cache clear was left behind.  This commit removes the cache clear.  The
    following commit shows the old clear and then read.
    
    
https://github.com/apache/accumulo/blob/ff8c875b83880c1fde3562f1280d6e20bc1ac682/server/master/src/main/java/org/apache/accumulo/master/tableOps/bulkVer2/PrepBulkImport.java#L89-L95
---
 .../org/apache/accumulo/manager/tableOps/bulkVer2/PrepBulkImport.java    | 1 -
 1 file changed, 1 deletion(-)

diff --git 
a/server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/PrepBulkImport.java
 
b/server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/PrepBulkImport.java
index 62e4315d87..4fa7d09b12 100644
--- 
a/server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/PrepBulkImport.java
+++ 
b/server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/PrepBulkImport.java
@@ -95,7 +95,6 @@ public class PrepBulkImport extends ManagerRepo {
     if (manager.onlineTabletServers().isEmpty()) {
       return 500;
     }
-    manager.getContext().clearTableListCache();
 
     return Utils.reserveHdfsDirectory(manager, bulkInfo.sourceDir, tid);
   }

Reply via email to