[GitHub] [hive] nrg4878 commented on a change in pull request #2246: HIVE-24448: Support case-sensitivity for tables in REMOTE database

[GitBox <gi...@apache.org>]

nrg4878 commented on a change in pull request #2246:
URL: https://github.com/apache/hive/pull/2246#discussion_r629562048



##########
File path: ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java
##########
@@ -135,6 +136,7 @@
   protected Context ctx;
   protected Map<String, String> idToTableNameMap;
   protected QueryProperties queryProperties;
+  protected static boolean isRemoteType = false;

Review comment:
       Why is this variable static? Wouldnt we have an issue if multiple queries were being processed at a time? 

##########
File path: ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java
##########
@@ -493,6 +495,10 @@ public static String getUnescapedUnqualifiedTableName(ASTNode node) throws Seman
     return getUnescapedName(node);
   }
 
+  public static boolean getIsRemoteType(){

Review comment:
       a) I think we should call this method isDbRemote() or isDatabaseRemote() instead of getIsRemoteType()
   b) this should be either a protected method so its only used by its sub-classes? being doesnt make much sense given the return value is specific to a database but it doesnt take any arguments. Or if we want to make it static, we can add a isDatabaseRemote(Database db) method to a utils function.

##########
File path: ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
##########
@@ -1096,7 +1096,7 @@ private String processTable(QB qb, ASTNode tabref) throws SemanticException {
 
     ASTNode tableTree = (ASTNode) (tabref.getChild(0));
 
-    String tabIdName = getUnescapedName(tableTree).toLowerCase();
+    String tabIdName = BaseSemanticAnalyzer.getIsRemoteType()?getUnescapedName(tableTree): getUnescapedName(tableTree).toLowerCase();

Review comment:
       if we make it an instance method, we will not need the classname to invoke it.




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org