Jason Fehr has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22511 )

Change subject: IMPALA-13772: Fix Workload Management DMLs Timeouts
......................................................................


Patch Set 6:

(1 comment)

> Patch Set 6:
>
> (1 comment)

http://gerrit.cloudera.org:8080/#/c/22511/6/tests/custom_cluster/test_query_log.py
File tests/custom_cluster/test_query_log.py:

http://gerrit.cloudera.org:8080/#/c/22511/6/tests/custom_cluster/test_query_log.py@776
PS6, Line 776: "--debug_actions=INTERNAL_SERVER_AFTER_SUBMIT:SLEEP@2000",
> Wonder if this can be replaced with default_query_options arg.
That is a good suggestion. I tried it out but ran into the problem that the 
default query options are not added at the level where the 
'INTERNAL_SERVER_AFTER_SUBMIT' DebugAction is executed (at 
https://gerrit.cloudera.org/c/22511/6/be/src/service/internal-server.cc#166).  
The default query actions are added somewhere in the SubmitQuery() call at 
https://gerrit.cloudera.org/c/22511/6/be/src/service/internal-server.cc#162



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0cc7fbce40eadfb253d8cff5cbb83e2ad63a979f
Gerrit-Change-Number: 22511
Gerrit-PatchSet: 6
Gerrit-Owner: Jason Fehr <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Comment-Date: Fri, 21 Feb 2025 19:24:56 +0000
Gerrit-HasComments: Yes

Reply via email to