Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24773 )
Change subject: IMPALA-15218 (addendum): Set trusted_jar_paths for all planner tests ...................................................................... IMPALA-15218 (addendum): Set trusted_jar_paths for all planner tests This moves the logic for setting the trusted_jar_paths from PlannerTest to PlannerTestBase. This fixes the S3PlannerTest for external data sources. Testing: - Ran S3 job Change-Id: I3a17c0a262683ca01d149743e8feafd8c5815594 Reviewed-on: http://gerrit.cloudera.org:8080/24773 Reviewed-by: Joe McDonnell <[email protected]> Tested-by: Michael Smith <[email protected]> --- M fe/src/test/java/org/apache/impala/planner/PlannerTest.java M fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java 2 files changed, 15 insertions(+), 12 deletions(-) Approvals: Joe McDonnell: Looks good to me, approved Michael Smith: Verified -- To view, visit http://gerrit.cloudera.org:8080/24773 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: branch-4.5.2 Gerrit-MessageType: merged Gerrit-Change-Id: I3a17c0a262683ca01d149743e8feafd8c5815594 Gerrit-Change-Number: 24773 Gerrit-PatchSet: 3 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]>
