[DRILL-5221] cancel message is delayed until queryid or data is received - ASF JIRA

Attach files

Attach Screenshot

Voters

Watch issue

Watchers

Create sub-task

Link

Clone

Update Comment Author

Replace String in Comment

Update Comment Visibility

Delete Comments

XML

Word

Printable

JSON

Details

Type: Improvement
Status: Resolved
Priority: Major
Resolution: Fixed
Affects Version/s: 1.9.0
Fix Version/s: 1.10.0
Component/s: Client - C++
Labels:
- ready-to-commit

Description

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 Venki Korukanti in his initial patch (https://github.com/vkorukanti/drill/commit/e0ef6349aac48de5828b6d725c2cf013905d18eb) but was omitted when I backported it post metadata changes.