(accumulo) branch 2.1 updated: Added better/consistent error messages when setting properties in Manager (#5789)

[dlmarion]

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

dlmarion 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 44b280b237 Added better/consistent error messages when setting 
properties in Manager (#5789)
44b280b237 is described below

commit 44b280b237a618b359ae46e3fe3e623e49a6795a
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Fri Aug 8 15:27:55 2025 -0400

    Added better/consistent error messages when setting properties in Manager 
(#5789)
---
 .../org/apache/accumulo/manager/ManagerClientServiceHandler.java   | 7 ++++---
 1 file changed, 4 insertions(+), 3 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 b0f173974b..4fc07fa8b0 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
@@ -287,7 +287,7 @@ public class ManagerClientServiceHandler implements 
ManagerClientService.Iface {
           TableOperation.SET_PROPERTY, TableOperationExceptionType.OTHER,
           "Error modifying table properties: tableId: " + tableId.canonical());
     } catch (IllegalArgumentException iae) {
-      throw new ThriftPropertyException();
+      throw new ThriftPropertyException("Modify properties", "failed", 
iae.getMessage());
     }
 
   }
@@ -455,7 +455,8 @@ public class ManagerClientServiceHandler implements 
ManagerClientService.Iface {
       SystemPropUtil.setSystemProperty(manager.getContext(), property, value);
     } catch (IllegalArgumentException iae) {
       Manager.log.error("Problem setting invalid property", iae);
-      throw new ThriftPropertyException(property, value, "Property is 
invalid");
+      throw new ThriftPropertyException(property, value,
+          "Property is invalid. message: " + iae.getMessage());
     } catch (Exception e) {
       Manager.log.error("Problem setting config property in zookeeper", e);
       throw new TException(e.getMessage());
@@ -517,7 +518,7 @@ public class ManagerClientServiceHandler implements 
ManagerClientService.Iface {
           TableOperation.SET_PROPERTY, TableOperationExceptionType.OTHER,
           "Error modifying namespace properties");
     } catch (IllegalArgumentException iae) {
-      throw new ThriftPropertyException("All properties", "failed", 
iae.getMessage());
+      throw new ThriftPropertyException("Modify properties", "failed", 
iae.getMessage());
     }
   }