jacobsimpson opened a new pull request, #7009:
URL: https://github.com/apache/arrow-rs/pull/7009

   # Which issue does this PR close?
   
   Closes #1706.
   
   # Rationale for this change
   
   I'd like to include the Flight service in the reflection service in one of 
my projects.
   
   # What changes are included in this PR?
   
   * Added generation of the necessary binary data to the protobuf compile 
steps.
   * Added constants representing the generated binary data in the compiled 
library.
   * Rustdoc for the new constants.
   * Added `tonic-reflection` to the dev dependencies so `cargo test --doc` and 
`cargo test --doc --features "flight-sql-experimental"` work.
   
   # Are there any user-facing changes?
   
   New constants should be visible to library consumers. Should be backwards 
compatible.
   
   # Testing
   
   Using a simple test service initialised using the reflection code in the 
rustdocs, `grpcurl` says:
   ```
   $ grpcurl --plaintext localhost:50051 list
   arrow.flight.protocol.FlightService
   grpc.reflection.v1.ServerReflection```
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to