drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From paul-rogers <...@git.apache.org>
Subject [GitHub] drill pull request #921: DRILL-4286 Graceful shutdown of drillbit
Date Tue, 14 Nov 2017 22:41:53 GMT
Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/921#discussion_r150985739
  
    --- Diff: protocol/src/main/java/org/apache/drill/exec/proto/beans/RpcType.java ---
    @@ -25,28 +25,8 @@
         HANDSHAKE(0),
         ACK(1),
         GOODBYE(2),
    -    RUN_QUERY(3),
    -    CANCEL_QUERY(4),
    -    REQUEST_RESULTS(5),
    -    RESUME_PAUSED_QUERY(11),
    -    GET_QUERY_PLAN_FRAGMENTS(12),
    -    GET_CATALOGS(14),
    -    GET_SCHEMAS(15),
    -    GET_TABLES(16),
    -    GET_COLUMNS(17),
    -    CREATE_PREPARED_STATEMENT(22),
    -    GET_SERVER_META(8),
    -    QUERY_DATA(6),
    -    QUERY_HANDLE(7),
    -    QUERY_PLAN_FRAGMENTS(13),
    -    CATALOGS(18),
    -    SCHEMAS(19),
    -    TABLES(20),
    -    COLUMNS(21),
    -    PREPARED_STATEMENT(23),
    -    SERVER_META(9),
    -    QUERY_RESULT(10),
    -    SASL_MESSAGE(24);
    +    REQ_RECORD_BATCH(3),
    +    SASL_MESSAGE(4);
    --- End diff --
    
    Is this your change? Why has the list of RPC types changed? Is this a merge issue?


---

Mime
View raw message