[jira] [Commented] (DRILL-5221) cancel message is delayed until queryid or data is received

["Laurent Goujon (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/DRILL-5221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15837076#comment-15837076 ] 

Laurent Goujon commented on DRILL-5221:
---------------------------------------

To clarify the description a bit, the current connector is waiting for the next message (queryId or batch data) to be received to send a cancellation message to the server.

What I am proposing is to send immediately the cancel message to the server if the queryId has already been received. Otherwise to wait until the queryId has been received.

> cancel message is delayed until queryid or data is received
> -----------------------------------------------------------
>
>                 Key: DRILL-5221
>                 URL: https://issues.apache.org/jira/browse/DRILL-5221
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Client - C++
>    Affects Versions: 1.9.0
>            Reporter: Laurent Goujon
>
> When user is calling the cancel method of the C++ client, the client wait for a message from the server to reply back with a cancellation message.
> In case of queries taking a long time to return batch results, it means cancellation won't be effective until the next batch is received, instead of cancelling right away the query (assuming the query id has already been received, which is generally the case).
> It seems this was foreseen by [~vkorukanti] in his initial patch (https://github.com/vkorukanti/drill/commit/e0ef6349aac48de5828b6d725c2cf013905d18eb) but was omitted when I backported it post metadata changes.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)