vikramahuja1001 commented on code in PR #4501:
URL: https://github.com/apache/hive/pull/4501#discussion_r1275491091
##########
ql/src/test/results/clientpositive/llap/temp_table_add_part_multiple.q.out:
##########
@@ -115,5 +115,5 @@ POSTHOOK: Input: default@add_part_test_n1_temp@ds=2010-04-01
#### A masked pattern was here ####
100 100 2010-01-01
200 200 2010-02-01
-500 400 2010-04-01
400 300 2010-03-01
+500 400 2010-04-01
Review Comment:
This change is only applicable for Temporary tables. Previously in the case
of older getPartitions API, we are directly fetching the partitions from the
PartitionTree(check ` SessionHiveMetaStoreClient. listPartitions(String
catName, String dbName, String tblName, int maxParts)`). But in the case of new
API involving batching, we first fetch the partitionNames from the
PartitionTree(check ` SessionHiveMetaStoreClient.listPartitionNames(String
catName, String dbName, String tblName,
int maxParts)` ). We then sort those partition names. Since we are
sorting the partitionNames we are essentially changing the order in which
partitions are fetched and added in the list.
--
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]
