chaijunjie0101 commented on code in PR #7117:
URL: https://github.com/apache/hbase/pull/7117#discussion_r2224179136


##########
hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncNonMetaRegionLocator.java:
##########
@@ -618,4 +635,48 @@ int getNumberOfCachedRegionLocations(TableName tableName) {
     return tableCache.regionLocationCache.getAll().stream()
       .mapToInt(RegionLocations::numNonNullElements).sum();
   }
+
+  private TimerTask getInvalidateMetaCacheTask(long 
metaCacheInvalidateInterval) {
+    return new TimerTask() {
+      @Override
+      public void run(Timeout timeout) throws Exception {
+        invalidateTableCache();
+        AsyncConnectionImpl.RETRY_TIMER.newTimeout(this, 
metaCacheInvalidateInterval,
+          TimeUnit.MILLISECONDS);
+      }
+    };
+  }
+
+  private void invalidateTableCache() {
+    // Should not throw any exception from here, it will affect schedule a new 
specified TimerTask.
+    Set<TableName> cachedTables = cache.keySet();
+    if (!cachedTables.isEmpty()) {
+      AsyncAdmin admin = conn.getAdmin();
+      for (TableName tableName : cachedTables) {

Review Comment:
   thanks for suggestting, call itself in the last table's callback logic in 
the latest PR.



-- 
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: issues-unsubscr...@hbase.apache.org

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

Reply via email to