Matt, Thanks for your support. I'll certainly look closely at DenseUnion and SparseUnion before implementing. I mentioned DenseUnion only because it is explicitly named in the FlightSQL proto definition, so it appears to be on the critical path to FlightSQL.
On Mon, Jun 6, 2022 at 11:48 AM Matt Topol <zotthewiz...@gmail.com> wrote: > Sounds good to me, I haven't had the full opportunity to put together the > union types lately in order to start working on FlightSQL. The only concern > I have is that it might make sense to try to come at it as building both > the Dense and Sparse unions together rather than building one first and > then the other (at least in approach). Something like maybe a `Union` > interface which both the Dense and Sparse union Array types implement etc. > > Anyway, let me know when you put a PR up and I'll definitely take a look > (or feel free to reach out if you run into any issues / need any > assistance). > > Thanks for taking this on! > > --Matt > > On Mon, Jun 6, 2022 at 2:44 PM Jacob Marble <jacobmar...@influxdata.com> > wrote: > >> (cc Matt Topol aka zeroshade) >> >> Good morning, >> >> I plan to start working on the FlightSQL client for Golang. My first task >> is to implement DenseUnion, as it is missing from the Golang Arrow library. >> >> Any concerns? >> >> -- >> Jacob Marble >> πΊπΈ πΊπ¦ >> > -- Jacob Marble πΊπΈ πΊπ¦
