hgromer commented on code in PR #6961:
URL: https://github.com/apache/hbase/pull/6961#discussion_r2075575143


##########
hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncRequestFutureImpl.java:
##########
@@ -783,8 +787,7 @@ private void receiveGlobalFailure(MultiAction rsActions, 
ServerName server, int
       // any of the regions in the MultiAction and do not update cache if 
exception is
       // from failing to submit action to thread pool
       if (clearServerCache) {
-        updateCachedLocations(server, regionName, row,
-          ClientExceptionsUtil.isMetaClearingException(t) ? null : t);
+        updateCachedLocations(server, regionName, row, t);

Review Comment:
   This also solves the frustration of seeing "UnknownException" when 
inspecting meta cache clear exceptions. This has made it quite difficult to 
track down what triggered the meta cache clear. 
   
   I think it's always better to provide more context than less. Even if an 
exception is meta cache clearing (though it will be now), I'd still prefer to 
know the exact exception type that cleared the meta cache. 



-- 
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