This is an automated email from the ASF dual-hosted git repository.

kturner pushed a commit to branch 2.1
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/2.1 by this push:
     new 6f9aba449c log exception on failure set table prop (#6057)
6f9aba449c is described below

commit 6f9aba449c8561e0ff0f3e581f81afc520ea4ca0
Author: Keith Turner <[email protected]>
AuthorDate: Wed Jan 14 18:07:41 2026 -0500

    log exception on failure set table prop (#6057)
---
 .../org/apache/accumulo/manager/ManagerClientServiceHandler.java    | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/server/manager/src/main/java/org/apache/accumulo/manager/ManagerClientServiceHandler.java
 
b/server/manager/src/main/java/org/apache/accumulo/manager/ManagerClientServiceHandler.java
index 4fc07fa8b0..57efa11cd7 100644
--- 
a/server/manager/src/main/java/org/apache/accumulo/manager/ManagerClientServiceHandler.java
+++ 
b/server/manager/src/main/java/org/apache/accumulo/manager/ManagerClientServiceHandler.java
@@ -579,10 +579,12 @@ public class ManagerClientServiceHandler implements 
ManagerClientService.Iface {
         }
         PropUtil.setProperties(manager.getContext(), 
TablePropKey.of(manager.getContext(), tableId),
             Map.of(property, value));
+      } else {
+        throw new UnsupportedOperationException("table operation:" + 
op.name());
       }
     } catch (IllegalStateException ex) {
-      log.warn("Invalid table property, tried to set: tableId: " + 
tableId.canonical() + " to: "
-          + property + "=" + value);
+      log.warn("Invalid table property, tried to {}: tableId: {} to: {}={}", 
op.name(),
+          tableId.canonical(), property, value, ex);
       // race condition... table no longer exists? This call will throw an 
exception if the table
       // was deleted:
       ClientServiceHandler.checkTableId(manager.getContext(), tableName, op);

Reply via email to