[jira] [Commented] (IMPALA-5216) Admission control queuing should be asynchronous

2018-09-10 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/IMPALA-5216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16609850#comment-16609850
 ] 

ASF subversion and git services commented on IMPALA-5216:
-

Commit ab6bd74ff73f52a3bd0247027bd82d977189ded6 in impala's branch 
refs/heads/master from [~bikram.sngh91]
[ https://git-wip-us.apache.org/repos/asf?p=impala.git;h=ab6bd74 ]

IMPALA-7516: Fix query location accounting

As a side-effect of IMPALA-5216, queries that were scheduled to be
executed but eventually got rejected or canceled before starting
execution (initializing the coordinator object) got added to the
'query_locations' map but never got removed. As a result, the queries
debug page would sometimes show queries running under the
"Query Locations" section even if there weren't. This patch fixes that
behavior.

Testing:
Added a custom cluster test.

Change-Id: I66c8e59299747df57c9f39db1cb1f46ff6bbab7e
Reviewed-on: http://gerrit.cloudera.org:8080/11372
Reviewed-by: Impala Public Jenkins 
Tested-by: Impala Public Jenkins 


> Admission control queuing should be asynchronous
> 
>
> Key: IMPALA-5216
> URL: https://issues.apache.org/jira/browse/IMPALA-5216
> Project: IMPALA
>  Issue Type: Sub-task
>  Components: Backend
>Reporter: Dan Hecht
>Assignee: Bikramjeet Vig
>Priority: Major
>  Labels: admission-control, resource-management
> Fix For: Impala 2.13.0, Impala 3.1.0
>
>
> Currently, admission control queuing occurs synchronously w.r.t. the 
> {{ExecuteStatement}} client RPC. That is, a query handle is not returned 
> until the query is admitted.
> Instead, the queuing should occur on the asynchronous path.  That way, the 
> client gets a query handle back immediately and can e.g. cancel a query that 
> is in the admission control queue.
> We'll also need a way to better expose the progress of a query handle 
> (related to IMPALA-124). E.g. that the query is queued for admission and what 
> resource(s) it's waiting on.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-all-unsubscr...@impala.apache.org
For additional commands, e-mail: issues-all-h...@impala.apache.org



[jira] [Commented] (IMPALA-5216) Admission control queuing should be asynchronous

2018-06-13 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/IMPALA-5216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16511338#comment-16511338
 ] 

ASF subversion and git services commented on IMPALA-5216:
-

Commit 2de9db8fc64dc1054ac3ca41ab0f0047db670da6 in impala's branch 
refs/heads/master from [~bikram.sngh91]
[ https://git-wip-us.apache.org/repos/asf?p=impala.git;h=2de9db8 ]

IMPALA-5216: Make admission control queuing async

Implement asynchronous admission control queuing. This is achieved by
running the admission control code-path in a separate thread. Major
changes include: propagating cancellation to the admission control
thread and dequeuing thread, and adding a new Query Operation State
called "PENDING" that represents the state between completion of
planning and starting of query execution.

Testing:
- Added a deterministic end to end test and a session expiry test.
- Ran multiple stress tests successfully with a cancellation probability
of 60% and with different values for the following parameters:
max_requests, queue_wait_timeout_ms. Ensured that the impalad was in a
valid state afterwards (no orphan fragments or wrong metrics).
- Ran all exhaustive tests and ASAN core tests successfully.
- Ran data load successfully.

Change-Id: I989cf5b259afb8f5bc5c35590c94961c81ce88bf
Reviewed-on: http://gerrit.cloudera.org:8080/10060
Reviewed-by: Tim Armstrong 
Tested-by: Impala Public Jenkins 


> Admission control queuing should be asynchronous
> 
>
> Key: IMPALA-5216
> URL: https://issues.apache.org/jira/browse/IMPALA-5216
> Project: IMPALA
>  Issue Type: Sub-task
>  Components: Backend
>Reporter: Dan Hecht
>Assignee: Bikramjeet Vig
>Priority: Major
>  Labels: admission-control, resource-management
>
> Currently, admission control queuing occurs synchronously w.r.t. the 
> {{ExecuteStatement}} client RPC. That is, a query handle is not returned 
> until the query is admitted.
> Instead, the queuing should occur on the asynchronous path.  That way, the 
> client gets a query handle back immediately and can e.g. cancel a query that 
> is in the admission control queue.
> We'll also need a way to better expose the progress of a query handle 
> (related to IMPALA-124). E.g. that the query is queued for admission and what 
> resource(s) it's waiting on.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-all-unsubscr...@impala.apache.org
For additional commands, e-mail: issues-all-h...@impala.apache.org