tanishq-chugh opened a new pull request, #5160: URL: https://github.com/apache/hive/pull/5160
What changes were proposed in this pull request? Modify the getVar(Configuration conf, ConfVars var) & getStringCollection(Configuration conf, ConfVars var) methods of the MetastoreConf class to use toString() method instead of type-casting. Why are the changes needed? When using the getVar(Configuration conf, ConfVars var) method of the MetastoreConf class, Apache breaks when e.g. trying to retrieve the environment variable "USE_SSL" and it isn't set in the system. The method then tries to cast the default value, which is the boolean false for USE_SSL, to a String which can't work. Also in the getStringCollection(Configuration conf, ConfVars var) method it tries to cast any default values to a String. Does this PR introduce any user-facing change? No Is the change a dependency upgrade? No How was this patch tested? Manual tests. After making the necessary changes, we build the hive project again. After manually testing some DDL and SQL commands, hive is working as expected. -- 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
