You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "keith-turner (via GitHub)" <gi...@apache.org> on 2023/04/25 14:33:14 UTC

[GitHub] [accumulo] keith-turner opened a new pull request, #3339: removes vestigial cache clear in bulk import

keith-turner opened a new pull request, #3339:
URL: https://github.com/apache/accumulo/pull/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


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [accumulo] keith-turner merged pull request #3339: removes vestigial cache clear in bulk import

Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner merged PR #3339:
URL: https://github.com/apache/accumulo/pull/3339


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [accumulo] ddanielr commented on pull request #3339: removes vestigial cache clear in bulk import

Posted by "ddanielr (via GitHub)" <gi...@apache.org>.
ddanielr commented on PR #3339:
URL: https://github.com/apache/accumulo/pull/3339#issuecomment-1522108469

   LGTM.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org