Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/23981 )
Change subject: IMPALA-15026: Upgrade to pytest 9.0.3 ...................................................................... Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/23981/8/tests/hs2/hs2_test_suite.py File tests/hs2/hs2_test_suite.py: http://gerrit.cloudera.org:8080/#/c/23981/8/tests/hs2/hs2_test_suite.py@133 PS8, Line 133: def setup_method(self): > Should setup_method and teardown_method also call the superclass methods? I think that is the main thing. This change was based on something from a few months ago before we added that assert. I can post a new patch set with that. -- To view, visit http://gerrit.cloudera.org:8080/23981 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4f06272f14a91d71a7701e1863f954e99aa4e994 Gerrit-Change-Number: 23981 Gerrit-PatchSet: 8 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Comment-Date: Thu, 21 May 2026 17:35:30 +0000 Gerrit-HasComments: Yes
