Csaba Ringhofer has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13414 )
Change subject: IMPALA-8369: Add HIVE_MAJOR_VERSION section to planner tests + some fixes ...................................................................... IMPALA-8369: Add HIVE_MAJOR_VERSION section to planner tests + some fixes Hive 3 creates different number of files for some tables than Hive2, which broke some test cases in resource-requirements.test. The fix is to run different versions of these tests depending on Hive version. This is done by adding a new section HIVE_MAJOR_VERSION, which leads to skipping the given test case if the Hive version is different in the cluster, e.g.: --- HIVE_MAJOR_VERSION 3 Change-Id: Ied7ba7911da23cbca12149e062f4e1a444613a36 Reviewed-on: http://gerrit.cloudera.org:8080/13414 Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Reviewed-by: Csaba Ringhofer <csringho...@cloudera.com> --- M fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java M fe/src/test/java/org/apache/impala/testutil/TestFileParser.java M testdata/workloads/functional-planner/queries/PlannerTest/resource-requirements.test 3 files changed, 154 insertions(+), 1 deletion(-) Approvals: Impala Public Jenkins: Verified Csaba Ringhofer: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/13414 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ied7ba7911da23cbca12149e062f4e1a444613a36 Gerrit-Change-Number: 13414 Gerrit-PatchSet: 5 Gerrit-Owner: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
