wecharyu commented on code in PR #5206:
URL: https://github.com/apache/hive/pull/5206#discussion_r1610289343
##########
ql/src/java/org/apache/hadoop/hive/ql/metadata/SessionHiveMetaStoreClient.java:
##########
@@ -1201,7 +1201,7 @@ public List<String> listPartitionNames(String catName,
String dbName, String tbl
public List<String> listPartitionNames(String catName, String dbName, String
tblName,
List<String> partVals, int maxParts) throws TException {
org.apache.hadoop.hive.metastore.api.Table table = getTempTable(dbName,
tblName);
- if (table == null) {
+ if (table == null || partVals == null) {
Review Comment:
> but i see that two of them are not changed
One is that `SessionHiveMetaStoreClient#istPartitionNamesRequest` does not
check null for part_vals, I think it's acceptable because
`HiveMetaStoreClient#listPartitionNamesRequest` also not check it.
The other is `SessionHiveMetaStoreClient#exchangePartitions` where part_vals
can not be null.
I can not find out the side effect on enable null or empty part_vals in
these calls if we do regard null or empty part_vals is used to fetch all parts
or part names.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
