[DRILL-2498] Separate QueryResult into two messages QueryResult and QueryData

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.7.0
• Fix Version/s: 0.9.0
• Component/s: Execution - Flow, Execution - RPC
• Assignee: Abdel Hakim Deneche
• Reporter: Abdel Hakim Deneche
• Created: 18/Mar/15 23:31
• Updated: 04/Apr/15 06:19
• Resolved: 04/Apr/15 06:19

Description

Currently, both the Foreman and the ScreenRoot use QueryResult as part of a QueryWritableBatch to send data and/or query status. We should split this into two separate messages one for the data that will be sent from the ScreenRoot and another for the queryStatus that will be sent from the Foreman

Issue Links

• contains: DRILL-2384 QueryState is not making it back to the client
• incorporates: DRILL-2521 Revert from protobuf 2.6 to 2.5
• is depended upon by: DRILL-2673 Update UserServer <==> UserClient RPC to handle handshake response better
• is depended upon by: DRILL-1984 Drillbit does not send the last RPC message when "limit" is appended to queries on hbase voter table
• is depended upon by: DRILL-2573 Update C++ Client to handle separation of QueryResult and QueryData
• is depended upon by: DRILL-1802 Change Fragment and Foreman state and cancellation to use full synchronous acknowledgement
• is depended upon by: DRILL-2430 Improve Error Propagation (Umbrella)