-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FlightRPC][C++][Go][Java] Add URI scheme to reuse connection #40345
Milestone
Comments
lidavidm
added a commit
to lidavidm/arrow
that referenced
this issue
Mar 4, 2024
Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org>
lidavidm
added a commit
that referenced
this issue
Mar 5, 2024
…on (#40084) ### Rationale for this change https://docs.google.com/document/d/1g9M9FmsZhkewlT1mLibuceQO8ugI0-fqumVAXKFjVGg/edit?usp=sharing ### What changes are included in this PR? Base implementation sans integration test ### Are these changes tested? Yes ### Are there any user-facing changes? No * GitHub Issue: #40345 Lead-authored-by: David Li <li.davidm96@gmail.com> Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org> Co-authored-by: Sutou Kouhei <kou@cozmixng.org> Signed-off-by: David Li <li.davidm96@gmail.com>
Issue resolved by pull request 40084 |
mapleFU
pushed a commit
to mapleFU/arrow
that referenced
this issue
Mar 7, 2024
…nnection (apache#40084) ### Rationale for this change https://docs.google.com/document/d/1g9M9FmsZhkewlT1mLibuceQO8ugI0-fqumVAXKFjVGg/edit?usp=sharing ### What changes are included in this PR? Base implementation sans integration test ### Are these changes tested? Yes ### Are there any user-facing changes? No * GitHub Issue: apache#40345 Lead-authored-by: David Li <li.davidm96@gmail.com> Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org> Co-authored-by: Sutou Kouhei <kou@cozmixng.org> Signed-off-by: David Li <li.davidm96@gmail.com>
thisisnic
pushed a commit
to thisisnic/arrow
that referenced
this issue
Mar 8, 2024
…nnection (apache#40084) ### Rationale for this change https://docs.google.com/document/d/1g9M9FmsZhkewlT1mLibuceQO8ugI0-fqumVAXKFjVGg/edit?usp=sharing ### What changes are included in this PR? Base implementation sans integration test ### Are these changes tested? Yes ### Are there any user-facing changes? No * GitHub Issue: apache#40345 Lead-authored-by: David Li <li.davidm96@gmail.com> Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org> Co-authored-by: Sutou Kouhei <kou@cozmixng.org> Signed-off-by: David Li <li.davidm96@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
https://lists.apache.org/thread/pc9fs0hf8t5ylj9os00r9vg8d2xv2npz
Add a way that services can explicitly indicate to clients that they may reuse the connection used to get FlightInfo to also fetch data, instead of requiring the service to somehow know or figure out its public address (which may be difficult when proxies, etc. are involved).
Component(s)
C++, FlightRPC, Go, Java
The text was updated successfully, but these errors were encountered: