This is an automated email from the ASF dual-hosted git repository. dlmarion pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/accumulo.git
commit cf42adff3f988ff467ff3f71449732b32703573f Merge: 0ff1b21f2e a47e4b8601 Author: Dave Marion <dlmar...@apache.org> AuthorDate: Tue Jan 7 20:47:07 2025 +0000 Merge branch '3.1' .../accumulo/tserver/TabletClientHandler.java | 6 +++++ .../org/apache/accumulo/tserver/tablet/Tablet.java | 27 +++++++++++----------- 2 files changed, 19 insertions(+), 14 deletions(-) diff --cc server/tserver/src/main/java/org/apache/accumulo/tserver/TabletClientHandler.java index 117d87a96d,f137a6df3b..1d13eff727 --- a/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletClientHandler.java +++ b/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletClientHandler.java @@@ -397,7 -450,11 +398,11 @@@ public class TabletClientHandler implem us.flushTime += (t2 - pt1); us.commitTimes.addStat(t2 - t1); - updateAvgCommitTime(t2 - t1, sendables.size()); + updateAvgCommitTime(t2 - t1, sendableMutations); + } catch (Exception e) { + TraceUtil.setException(span3, e, true); + log.error("Error committing mutations sent from {}", TServerUtils.clientAddress.get(), e); + throw e; } finally { span3.end(); } @@@ -657,9 -698,10 +662,10 @@@ long t1 = System.currentTimeMillis(); sendables.forEach(CommitSession::commit); long t2 = System.currentTimeMillis(); - updateAvgCommitTime(t2 - t1, sendables.size()); + updateAvgCommitTime(t2 - t1, sendableMutations); } catch (Exception e) { TraceUtil.setException(span3, e, true); + log.error("Error committing mutations sent from {}", TServerUtils.clientAddress.get(), e); throw e; } finally { span3.end();