mszurap opened a new pull request, #3950:
URL: https://github.com/apache/hive/pull/3950
### What changes were proposed in this pull request?
In this jira proposing:
- adding a new config "hive.conf.ignored.variable.list"
- accepts strings, comma separated list of variables (made a small change to
parse values in the same way as it is with HIVE_CONF_HIDDEN_LIST)
- the config is empty by default, it can be set in hive-site.xml only
- adding the "hive.conf.ignored.variable.list" to the restricted list
("hive.conf.restricted.list") internally - so it cannot be modified during
runtime
- adding tests for the changes
### Why are the changes needed?
In certain environments (usually after upgrades) we need to restrict users
from changing some Hive configurations during runtime, for example
"mapreduce.job.queuename" or "hive.execution.engine" should not be changed in
an environment, however notifying all users and asking them to change all of
their jobs is not possible.
### Does this PR introduce _any_ user-facing change?
- Yes, new config "hive.conf.ignored.variable.list" which may need to be
documented.
- Trying to set such an ignored variable will result in that the "set"
request is ignored and the user gets
> Configuration xxxx is ignored and cannot be set during runtime.
instead of the "No rows affected" (successful) setting.
### How was this patch tested?
- Added a new unit test TestSetProcessorWithIgnoreList to test this feature
- Started a local HS2 instance with a modified "hive-site.xml" to test if
the setting is populated properly and works as above explained
--
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: gitbox-unsubscr...@hive.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
