This is an automated email from the ASF dual-hosted git repository. dlmarion pushed a commit to branch elasticity in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/elasticity by this push: new 995f318459 Removed TODO in TabletRefresher (#4417) 995f318459 is described below commit 995f318459efe6e1c3b29cf8fc9c0a27302f04dc Author: Dave Marion <dlmar...@apache.org> AuthorDate: Fri Apr 5 16:13:40 2024 -0400 Removed TODO in TabletRefresher (#4417) Other exceptions which could be more serious than TException would be raised as an ExecutionException when get() is called on the Future and would result in a RuntimeException being raised from TabletRefresher.refreshTablets. If a TException is thrown, then the refresh for all of the Tablets will be retried unless the TabletServer is no longer online. --- .../org/apache/accumulo/manager/tableOps/bulkVer2/TabletRefresher.java | 3 --- 1 file changed, 3 deletions(-) diff --git a/server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/TabletRefresher.java b/server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/TabletRefresher.java index a3d341a12b..cb963e583a 100644 --- a/server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/TabletRefresher.java +++ b/server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/TabletRefresher.java @@ -171,9 +171,6 @@ public class TabletRefresher { } catch (TException ex) { log.debug("rpc failed server: " + location + ", " + logId + " " + ex.getMessage(), ex); - // ELASTICITY_TODO are there any other exceptions we should catch in this method and check if - // the tserver is till alive? - // something went wrong w/ RPC return all extents as unrefreshed return refreshes; } finally {