You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "PANKAJ9768 (via GitHub)" <gi...@apache.org> on 2024/08/13 06:29:48 UTC
[I] Reconcile header size limit in flight server with flight client [arrow]
PANKAJ9768 opened a new issue, #43667:
URL: https://github.com/apache/arrow/issues/43667
### Describe the bug, including details regarding any error messages, version, and platform.
Flight server side header acceptance limit is 2kb whereas client can send header size upto Integer.MAX_VALUE resulting in request failure .
In flight client builder we can set header size limit but this is missing in flight server builder.
Also default size at client side is set to Integer.MAX_VALUE which allows client to send bigger payload in header but server side request is getting failed and no option to modify default size
Client side config :
https://github.com/apache/arrow/blob/main/java/flight/flight-core/src/main/java/org/apache/arrow/flight/FlightClient.java#L777C1-L778C1
https://github.com/apache/arrow/blob/main/java/flight/flight-core/src/main/java/org/apache/arrow/flight/FlightClient.java#L805
Server side :
https://github.com/apache/arrow/blob/main/java/flight/flight-core/src/main/java/org/apache/arrow/flight/FlightServer.java#L224
### Component(s)
FlightRPC, Java
--
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
Re: [I] Reconcile header size limit in flight server with flight client [arrow]
Posted by "PANKAJ9768 (via GitHub)" <gi...@apache.org>.
PANKAJ9768 commented on issue #43667:
URL: https://github.com/apache/arrow/issues/43667#issuecomment-2288055832
> Agreed. A PR would be welcome.
Yup, Will raise it soon
--
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
Re: [I] [Java] Reconcile header size limit in flight server with flight client [arrow]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm closed issue #43667: [Java] Reconcile header size limit in flight server with flight client
URL: https://github.com/apache/arrow/issues/43667
--
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
Re: [I] Reconcile header size limit in flight server with flight client [arrow]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #43667:
URL: https://github.com/apache/arrow/issues/43667#issuecomment-2288045515
Agreed. A PR would be welcome.
--
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
Re: [I] [Java] Reconcile header size limit in flight server with flight client [arrow]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #43667:
URL: https://github.com/apache/arrow/issues/43667#issuecomment-2311341209
Issue resolved by pull request 43697
https://github.com/apache/arrow/pull/43697
--
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
Re: [I] Reconcile header size limit in flight server with flight client [arrow]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #43667:
URL: https://github.com/apache/arrow/issues/43667#issuecomment-2285446196
I would recommend just configuring the underlying gRPC server directly instead of relying on Flight defaults (which are mostly fairly arbitrary).
--
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
Re: [I] Reconcile header size limit in flight server with flight client [arrow]
Posted by "PANKAJ9768 (via GitHub)" <gi...@apache.org>.
PANKAJ9768 commented on issue #43667:
URL: https://github.com/apache/arrow/issues/43667#issuecomment-2288030207
Yeah, We can do that , But still if we can keep Flight default values in sync in both client and server side then it would be convenient
--
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