You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "lidavidm (via GitHub)" <gi...@apache.org> on 2023/07/17 13:00:10 UTC
[GitHub] [arrow] lidavidm opened a new issue, #36722: [FlightRPC] Explore using HTTP status codes as the canonical status definition
lidavidm opened a new issue, #36722:
URL: https://github.com/apache/arrow/issues/36722
### Describe the enhancement requested
We currently ad-hoc map through some of gRPC's status codes and awkwardly map to Arrow status codes (in C++) or a custom exception (in Java). Ideally we'd always expose exactly a consistent set of Flight status codes (this makes it easier to predict behavior across languages, at maybe the cost of a little convenience for someone only working in a single language)
Also see
- #34544
- #35888
- https://github.com/grpc/grpc/blob/master/doc/http-grpc-status-mapping.md
- https://github.com/cloudendpoints/esp/blob/1b02c011f071a8d400dbbbc08b512700b69ae519/src/api_manager/utils/status.cc#L287-L327
- https://github.com/apache/arrow/pull/36517?notification_referrer_id=NT_kwDOAAUA77E2OTg3NTI3MDg4OjMyNzkxOQ#discussion_r1265306369
### Component(s)
FlightRPC
--
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: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] lidavidm closed issue #36722: [FlightRPC] Explore using HTTP status codes as the canonical status definition
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm closed issue #36722: [FlightRPC] Explore using HTTP status codes as the canonical status definition
URL: https://github.com/apache/arrow/issues/36722
--
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: issues-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] lidavidm commented on issue #36722: [FlightRPC] Explore using HTTP status codes as the canonical status definition
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #36722:
URL: https://github.com/apache/arrow/issues/36722#issuecomment-1686532441
[Googleapis already defines canonical mappings between gRPC and HTTP status codes](https://buf.build/googleapis/googleapis/file/main:google/rpc/code.proto). However, these are lossy in both directions. Given that, I'd rather stick with the canonical gRPC codes for Flight since we already use them, and we should just update the Flight codes to be 1:1 with gRPC (e.g. to fix #35888).
We could define our own symmetric mapping, but I'd rather not risk doing something that would confuse people who also want to use existing gRPC infrastructure (e.g. HTTP proxies).
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org