ashish-kumar-sharma commented on a change in pull request #2211: URL: https://github.com/apache/hive/pull/2211#discussion_r626394973
########## File path: ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java ########## @@ -5838,10 +5838,20 @@ public void dropConstraint(String dbName, String tableName, String constraintNam } } - public SQLAllTableConstraints getTableConstraints(String dbName, String tblName) + public SQLAllTableConstraints getTableConstraints(String dbName, String tblName, long tableId) throws HiveException, NoSuchObjectException { try { - return getMSC().getAllTableConstraints(new AllTableConstraintsRequest(dbName, tblName, getDefaultCatalog(conf))); + + ValidWriteIdList validWriteIdList = null; Review comment: Make sense. Actually there already exist a method created by you for same. I will replace this code with getValidWriteIdList(String dbName, String tableName). -- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org