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 <[email protected]>
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 {
