Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/22731 )
Change subject: IMPALA-13930: ImpylaHS2Connection should only open cursor as needed ...................................................................... Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/22731/6//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/22731/6//COMMIT_MSG@9 PS6, Line 9: Before this patch, ImpylaHS2Connection unconditionally opened a : cursor (and HS2 session) as it connected, followed by running a "SET : ALL" que > Could we slightly change this paragraph to the following so that it would b Done http://gerrit.cloudera.org:8080/#/c/22731/6//COMMIT_MSG@20 PS6, Line 20: Fix test_query_profile_contains_query_compilation_metadata_cached_event : slightly by setting the 'sync_ddl' option because the test is flaky : without it. > It's just for my own understanding. Nothing has to be changed here. I just deflake it along with this patch. Without this patch, It should be possible to reproduce the flakiness with sufficient loop. -- To view, visit http://gerrit.cloudera.org:8080/22731 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9d5e3e5c11ad386b7202431201d1a4cff46cbff5 Gerrit-Change-Number: 22731 Gerrit-PatchSet: 8 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Fang-Yu Rao <fangyu....@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Yida Wu <wydbaggio...@gmail.com> Gerrit-Comment-Date: Thu, 10 Apr 2025 22:35:49 +0000 Gerrit-HasComments: Yes
