Uh oh!
There was an error while loading. Please reload this page.
[WIP] ARROW-8601: [Go][Flight] Added implementation of FlightDataWriter - #6731
[WIP] ARROW-8601: [Go][Flight] Added implementation of FlightDataWriter#6731stuartcarnie wants to merge 1 commit into
Conversation
stuartcarnie
commented
Mar 26, 2020
@sbinet do you know if there is any interest in moving this forward? |
sbinet
commented
Mar 26, 2020
not from my end, but happy to review it nonetheless. |
mindhash
commented
Apr 17, 2020
I am looking to try this out. Might as well create examples similar to https://github.com/apache/arrow/tree/master/java/flight/flight-core/src/main/java/org/apache/arrow/flight/example @sbinet should I wait for merge before adding anything new? |
| syntax = "proto3"; | ||
| option java_package = "org.apache.arrow.flight.impl"; | ||
| option go_package = "ipc"; |
There was a problem hiding this comment.
Can you use flight namespace, there's already a component named ipc in the C++ code base and this could bring confusion.
fsaintjacques
commented
Apr 27, 2020
Created https://jira.apache.org/jira/browse/ARROW-8601 for this |
yonidavidson
commented
Apr 27, 2020
Hi All, |
I would be very interested in using this if it moves forward.. (Native Go, and from WASM) |
emkornfield
commented
Sep 6, 2020
@sbinet did you ever take a look at this? Seems this has stalled? |
zeroshade
commented
Sep 11, 2020
I took a stab at building this out and adding more to it but I'm not sure what the etiquette here would be. Should I make a new PR or should I try pushing to this PR? for reference here's my branch: zeroshade@0fad81c |
stuartcarnie
commented
Sep 11, 2020
@zeroshade I have no problem if you want to create a new PR to take over this work. I would love to see something functional land in the Go implementation. |
emkornfield
commented
Sep 14, 2020
Closing this out as it appears #8175 supercedes it. |
Built out from #6731 with some inspiration from the existing Reader/Writer and the C++ Flight implementation. Still need to build out the tests some more, but would like to get opinions and thoughts on what I've got so far as I continue to build it out. Closes#8175 from zeroshade/zeroshade/go/flight Authored-by: Matthew Topol <mtopol@factset.com> Signed-off-by: Wes McKinney <wesm@apache.org>
Built out from apache/arrow#6731 with some inspiration from the existing Reader/Writer and the C++ Flight implementation. Still need to build out the tests some more, but would like to get opinions and thoughts on what I've got so far as I continue to build it out. Closes#8175 from zeroshade/zeroshade/go/flight Authored-by: Matthew Topol <mtopol@factset.com> Signed-off-by: Wes McKinney <wesm@apache.org>
Built out from apache/arrow#6731 with some inspiration from the existing Reader/Writer and the C++ Flight implementation. Still need to build out the tests some more, but would like to get opinions and thoughts on what I've got so far as I continue to build it out. Closes #8175 from zeroshade/zeroshade/go/flight Authored-by: Matthew Topol <mtopol@factset.com> Signed-off-by: Wes McKinney <wesm@apache.org>
DO NOT MERGE
I wanted make the initial code I wrote to support Apache Flight in Go available to others. We may be able to collaborate and move this forward.
I had success using this with InfluxDB and pandas.