zabetak opened a new pull request, #5897: URL: https://github.com/apache/hive/pull/5897
### What changes were proposed in this pull request? 1. Move deprecated property registation in the existing static block 2. Drop constructors and variables related to deprecated keys in ConfVars 3. Drop DEPRECATED_TEST_ENTRY that is only useful for testing purposes and adapt existing tests ### Why are the changes needed? Improve performance & code maintenance. Looping through ~400 ConfVars enumeration entries on every creation of a new Configuration entry is resource wasteful and unecessary. The deprecated properties are static and known at compile time so there is no reason to search and register them on every creation of a new Configuration instance. In addition, the presence of deprecation information inside the ConfVars enumeration leads to boilerplate code that is not really necessary. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Existing tests -- 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
