Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24065 )
Change subject: IMPALA-14776 (part 1 followup): Fix TestParquetMaxPageHeader ...................................................................... IMPALA-14776 (part 1 followup): Fix TestParquetMaxPageHeader Part 1 of IMPALA-14776 added a call to super.teardown_method() for TestParquetMaxPageHeader. However, it got the order wrong. The super().teardown_method() needs to come after TestParquetMaxPageHeader's own teardown steps, as those steps need the built-in clients. This moves the super().teardown_method() call to be after those steps. Testing: - Ran TestParquetMaxPageHeader Change-Id: If66b82d7386cdf748674a5a146d6b2b31471331d Reviewed-on: http://gerrit.cloudera.org:8080/24065 Reviewed-by: Michael Smith <[email protected]> Tested-by: Michael Smith <[email protected]> --- M tests/custom_cluster/test_parquet_max_page_header.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Smith: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/24065 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If66b82d7386cdf748674a5a146d6b2b31471331d Gerrit-Change-Number: 24065 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
