Github user sohami commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1203#discussion_r182507182
  
    --- Diff: protocol/src/main/protobuf/Coordination.proto ---
    @@ -18,6 +18,7 @@ message DrillbitEndpoint{
           OFFLINE = 3;
       }
       optional State state = 7;
    +  optional int32 http_port = 8;
    --- End diff --
    
    Please compile protobuf for c++ client as well (Instructions are 
[here](https://github.com/apache/drill/blob/master/contrib/native/client/readme.linux)).
 It would be really helpful if you can put a note in [readme.txt of this 
package](https://github.com/apache/drill/blob/master/protocol/readme.txt) for 
future dev to compile c++ client protobufs whenever a change is made in any of 
.proto files.


---

Reply via email to