jlalwani-amazon commented on code in PR #37:
URL: 
https://github.com/apache/flink-connector-hive/pull/37#discussion_r3313062722


##########
flink-connector-hive/src/main/java/org/apache/flink/table/planner/delegation/hive/copy/HiveParserQB.java:
##########
@@ -76,7 +75,7 @@ public class HiveParserQB {
      */
     private int numSubQueryPredicates;
 
-    private CreateTableDesc createTableDesc;
+    private Object createTableDesc;

Review Comment:
   This is similar to `PrincipalDesc` change in HiveShim. Hive4 moved the 
class,  I'm not sure where this is being set, but I'm making an assumption that 
whoever sets it will put Hive4's CreateTableDesc. So, this property needs to be 
able to accept both classes
   



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to