EdColeman opened a new pull request, #3680:
URL: https://github.com/apache/accumulo/pull/3680
Adds a prompt y/n when setting a deprecated property with the shell. Also
adds --force as an option to bypass the prompt.
Working with a user, we set a deprecated property - the consequence was that
the tserver logs started to be spammed with warning that a deprecated property
was set. If we had not been actively monitoring the monitor logs it likely
would have gone unnoticed for quite a while. The noisy log behavior has been
stated as desirable. With this change, shell users need to consciously set the
deprecated property by answering a y/n prompt, or using the --force option.
The prompt looks like
```
Trying to set deprecated property `tserver.compaction.major.concurrent.max`
continue (yes|no)? no
2023-08-08T17:56:11,361 [shell.Shell] ERROR:
org.apache.accumulo.core.util.BadArgumentException: Tried to set deprecated
property and force not specified. near index 10
```
--
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]
