[
https://issues.apache.org/jira/browse/DRILL-4015?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15013947#comment-15013947
]
ASF GitHub Bot commented on DRILL-4015:
---------------------------------------
Github user jacques-n commented on a diff in the pull request:
https://github.com/apache/drill/pull/263#discussion_r45370924
--- Diff: protocol/src/main/protobuf/User.proto ---
@@ -27,6 +27,7 @@ enum RpcType {
RESP_FUNCTION_LIST = 9;
QUERY_RESULT = 10; // drillbit is reporting a query status change, most
likely a terminal message, to the user
+ QUERY_WARNING = 12; // drillbit is sending a query warning to the user
--- End diff --
We actually started with only a compound result/warning (and that is also
included in this patch). One of the things we want to give is the user should
have an ability to prematurely terminate a query if they don't like the warning
that we are returning. (For example: we had to create a null field since it
didn't exist.) To do that, we may have an early warning that we want to return
out of band. For backwards compatibility, we can quell sending these back to
the user if they are on an older rpc version.
One question here: we should make the server decide whether the rpc version
number is okay (if we don't already). That way we can upgrade server and manage
backwards compatibility. If the client is currently checking the RPC version
and also terminating if it doesn't like it, we're screwed for backwards
compatbility. (My thought is that we should update version number whenever the
protocol changes but we need to relax the compatibility check. We can simply
decide what is cross-compatible in the code layer. (e.g. v3 works with v2
client but don't send them warnings.)
> Update DrillClient and JDBC driver to expose warnings provided via RPC layer
> ----------------------------------------------------------------------------
>
> Key: DRILL-4015
> URL: https://issues.apache.org/jira/browse/DRILL-4015
> Project: Apache Drill
> Issue Type: Sub-task
> Components: Client - JDBC, Execution - RPC
> Reporter: Jacques Nadeau
> Assignee: Abhijit Pol
> Fix For: 1.4.0
>
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
