ddanielr opened a new pull request, #3146: URL: https://github.com/apache/accumulo/pull/3146
relates to #3145 The shell config command was only checking Property Key Types and not Value Formats on Set operations. Changed Set operation to also validate value types to ensure proper format before setting properties. This changes the shell client behavior to now throw an error message to the user on an invalid property setting. Boolean Example: ``` root@uno> config -s tserver.port.search=123 2022-12-30T15:14:21,363 [shell.Shell] ERROR: org.apache.accumulo.core.util.BadArgumentException: Invalid Property value (requires type: boolean) near index 30 config -s tserver.port.search=123 ``` Port Example: ``` root@uno> config -s tserver.port.client=65539 2022-12-30T15:15:29,328 [shell.Shell] ERROR: org.apache.accumulo.core.util.BadArgumentException: Invalid Property value (requires type: port) near index 30 config -s tserver.port.client=65539 ``` This PR is staged against main, but we may want to backport this change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
