Vihang Karajgaonkar has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/17812 )
Change subject: IMPALA-10888: getPartitionsByNames should return sorted partitions ...................................................................... IMPALA-10888: getPartitionsByNames should return sorted partitions The catalog's HMS API implementation for getPartitionsByNames currently does not return partitions in any particular order. This can cause test flakiness where some tests expect partitions in a given order. Instead of modifying the tests this change changes the implementation of the API such that it always returns the partitions sorted by partition name. This is in-line with how hive metastore returns the partitions over this API. Testing: 1. Added a new unit test and modified existing test to assert the sorted order. Change-Id: I75aa9d714e888ec7f7efd62e475ba1d3a3342256 --- M fe/src/main/java/org/apache/impala/catalog/CatalogHmsAPIHelper.java M fe/src/test/java/org/apache/impala/catalog/metastore/CatalogHmsFileMetadataTest.java 2 files changed, 59 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/12/17812/2 -- To view, visit http://gerrit.cloudera.org:8080/17812 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I75aa9d714e888ec7f7efd62e475ba1d3a3342256 Gerrit-Change-Number: 17812 Gerrit-PatchSet: 2 Gerrit-Owner: Vihang Karajgaonkar <vih...@cloudera.com> Gerrit-Reviewer: Aman Sinha <amsi...@cloudera.com> Gerrit-Reviewer: Anonymous Coward <kis...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Sourabh Goyal <soura...@cloudera.com> Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com>