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

• 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
• Assignee: Laurent Goujon
• Reporter: Laurent Goujon
• Reviewer: Khurram Faraaz
• Created: 25/Jan/17 01:37
• Updated: 27/Mar/17 21:31
• Resolved: 02/Mar/17 22:49

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

Issue Links

• links to: GitHub Pull Request #733