jcamachor commented on a change in pull request #741: HIVE-22046
URL: https://github.com/apache/hive/pull/741#discussion_r310751284
 
 

 ##########
 File path: 
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
 ##########
 @@ -3089,7 +3089,7 @@ public Table get_table(final String dbname, final String 
name) throws MetaExcept
         NoSuchObjectException {
       String[] parsedDbName = parseDbName(dbname, conf);
       return getTableInternal(
-            parsedDbName[CAT_NAME], parsedDbName[DB_NAME], name, null, null, 
false);
+            parsedDbName[CAT_NAME], parsedDbName[DB_NAME], name, null, null, 
false, null);
 
 Review comment:
   Started making this change and there are already some boolean in the 
metastore thrift API that probably we would need to deprecate. To not make this 
patch any more complex, I just documented the API properly and added a 
precondition at the start of the method.

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to