Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22853 )

Change subject: IMPALA-14028: Refactor cancel_query_and_validate_state with 
Impyla
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22853/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22853/1//COMMIT_MSG@20
PS1, Line 20: 2. Make client operation such as execute_async, fetch, cancel,
            :    get_runtime_profile mutually exclusive.
            : 3. Do not attempt to fetch if there is a debug action set to 
prevent it
            :    (ie., wait GetNext indefinitely).
> I don't understand the tests completely, but isn't it actually  goal to tes
I tried, but Impyla fetch seems to just hang sometimes, presumably due to wait 
mechanism inside Impyla code.
Maybe both fetch and cancel client should be MinimalHS2Connection. I'll try 
again.



--
To view, visit http://gerrit.cloudera.org:8080/22853
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I626a1a06eb3d5dc9737c7d4289720e1f52d2a984
Gerrit-Change-Number: 22853
Gerrit-PatchSet: 1
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Xuebin Su <[email protected]>
Gerrit-Comment-Date: Mon, 05 May 2025 13:33:13 +0000
Gerrit-HasComments: Yes

Reply via email to