Hello Alex Behm, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10410 to look at the new patch set (#4). 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 --- 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(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/10/10410/4 -- 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: newpatchset Gerrit-Change-Id: I91e3e416ec6242fbf22d9f566fdd1ce225cb16ac Gerrit-Change-Number: 10410 Gerrit-PatchSet: 4 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>