Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/22847 )
Change subject: IMPALA-12201: Stabilize TestFetch ...................................................................... Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/22847/3/tests/query_test/test_fetch.py File tests/query_test/test_fetch.py: http://gerrit.cloudera.org:8080/#/c/22847/3/tests/query_test/test_fetch.py@146 PS3, Line 146: finally: : client.close_query(handle) : : runtime_profile = client.get_runtime_profile(handle) : : summary_stats = get_time_summary_stats_counter("ClientFetchWaitTimeStats", : runtime > I tried this out locally, and for me, it works to fetch the runtime profile You're right, it actually works! Maybe MinimalHS2Connection is much simpler than ImpylaHS2Connection that it is possible to do so. Impyla refuse to pull profile if query handle already closed. -- To view, visit http://gerrit.cloudera.org:8080/22847 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I52651df37a318357711d26d2414e025cce4185c3 Gerrit-Change-Number: 22847 Gerrit-PatchSet: 4 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Xuebin Su <x...@cloudera.com> Gerrit-Comment-Date: Fri, 02 May 2025 21:21:40 +0000 Gerrit-HasComments: Yes
