Skip to content

Commit 809b2f9

Browse files
authored
HIVE-23931: Addendum: ValidWriteIdList isn't used by get_*_constraints HMS APIs (#6135)
1 parent 55d9ab7 commit 809b2f9

File tree

8 files changed

+16
-197
lines changed

8 files changed

+16
-197
lines changed

ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6640,10 +6640,8 @@ public List<SQLCheckConstraint> getCheckConstraintList(String dbName, String tbl
66406640
public SQLAllTableConstraints getTableConstraints(String dbName, String tblName, long tableId)
66416641
throws HiveException, NoSuchObjectException {
66426642
try {
6643-
ValidWriteIdList validWriteIdList = getValidWriteIdList(dbName, tblName);
66446643
AllTableConstraintsRequest request = new AllTableConstraintsRequest(dbName, tblName, getDefaultCatalog(conf));
66456644
request.setTableId(tableId);
6646-
request.setValidWriteIdList(validWriteIdList != null ? validWriteIdList.writeToString() : null);
66476645
return getMSC().getAllTableConstraints(request);
66486646
} catch (NoSuchObjectException e) {
66496647
throw e;
@@ -6658,10 +6656,7 @@ public TableConstraintsInfo getTableConstraints(String dbName, String tblName, b
66586656
perfLogger.perfLogBegin(CLASS_NAME, PerfLogger.HIVE_GET_TABLE_CONSTRAINTS);
66596657

66606658
try {
6661-
6662-
ValidWriteIdList validWriteIdList = getValidWriteIdList(dbName,tblName);
66636659
AllTableConstraintsRequest request = new AllTableConstraintsRequest(dbName, tblName, getDefaultCatalog(conf));
6664-
request.setValidWriteIdList(validWriteIdList != null ? validWriteIdList.writeToString() : null);
66656660
request.setTableId(tableId);
66666661

66676662
SQLAllTableConstraints tableConstraints = getMSC().getAllTableConstraints(request);

standalone-metastore/metastore-common/src/gen/thrift/gen-cpp/hive_metastore_types.cpp

Lines changed: 1 addition & 23 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

standalone-metastore/metastore-common/src/gen/thrift/gen-cpp/hive_metastore_types.h

Lines changed: 1 addition & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)