Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10410 )
Change subject: IMPALA-7025: ignore resources in some planner test ...................................................................... IMPALA-7025: ignore resources in some planner test The issue was that the tablesample test verified the mem-estimate number, which depends on file sizes, which can vary slightly between data loads. Instead of trying to tweak the test to avoid the issue, instead provide a mechanism to ignore the exact values of resources in planner tests where they are not significant. Testing: Manually modified some values in tablesample.test, made sure that the test still passed. Manually modified the partition count in the expected output, made sure that the test failed. Change-Id: I91e3e416ec6242fbf22d9f566fdd1ce225cb16ac Reviewed-on: http://gerrit.cloudera.org:8080/10410 Reviewed-by: Tim Armstrong <tarmstr...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M fe/src/test/java/org/apache/impala/planner/PlannerTest.java M fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java M fe/src/test/java/org/apache/impala/testutil/TestUtils.java 3 files changed, 138 insertions(+), 86 deletions(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/10410 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I91e3e416ec6242fbf22d9f566fdd1ce225cb16ac Gerrit-Change-Number: 10410 Gerrit-PatchSet: 6 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>