You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by dl...@apache.org on 2025/08/08 19:28:01 UTC
(accumulo) branch 2.1 updated: Added better/consistent error messages when setting properties in Manager (#5789)
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 <dl...@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());
}
}