deniskuzZ commented on code in PR #5951:
URL: https://github.com/apache/hive/pull/5951#discussion_r2198170871


##########
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:
##########
@@ -7026,15 +7033,15 @@ public static void setHiveSiteLocation(URL location) {
   }
 
   public static void setHivemetastoreSiteUrl(URL location) {

Review Comment:
   it's not used anywhere, just 3 tests. Try searching through Apache projects 
on GitHub. I only came across an older version of the Flink connector, and it 
doesn’t appear to be in recent one.
   if you want to play safe, we can deprecate and add a new method with valid 
name.



-- 
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

Reply via email to