Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8113 )
Change subject: IMPALA-5966: Fix the result file location of PlannerTest ...................................................................... IMPALA-5966: Fix the result file location of PlannerTest The Plannertest result files should be written to $IMPALA_FE_TEST_LOGS_DIR/PlannerTest, but instead they are written to $IMPALA_FE_TEST_LOGS_DIR with "PlannerTest" as the prefix of their filenames. This patch fixes this bug and refactores surrounding code a little, replacing some Strings with Paths. Change-Id: I20005bd0421e1bb9f3eedbb003c97d92a8faf110 Reviewed-on: http://gerrit.cloudera.org:8080/8113 Reviewed-by: Alex Behm <alex.b...@cloudera.com> Tested-by: Impala Public Jenkins --- M fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java 1 file changed, 9 insertions(+), 11 deletions(-) Approvals: Alex Behm: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8113 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I20005bd0421e1bb9f3eedbb003c97d92a8faf110 Gerrit-Change-Number: 8113 Gerrit-PatchSet: 3 Gerrit-Owner: Tianyi Wang <tw...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tianyi Wang <tw...@cloudera.com>