Re: Review request for Dataset Java API PRs

2021-08-22 Thread Hongze Zhang
Hello guys, sorry I have to to request for review here again since progress didn't seem to be made yet :(. These PRs are important to Dataset Java implementation as the first version of it was too basic to serve advanced use cases. If reviewing of the big write support PR[1] sounds to be a bit of c

Re: Flight SQL

2021-08-22 Thread Micah Kornfield
In the interest of avoiding bit-rot I'm OK with checking in the PR once its been reviewed, but we probably shouldn't do an official release of it (I'm not sure if this is feasible with our current release scripts). On Sun, Aug 22, 2021 at 2:46 PM David Li wrote: > Sorry, you're right - I got the

Re: Flight SQL

2021-08-22 Thread David Li
Sorry, you're right - I got the process mixed up. Having two implementations ready before a vote was discussed a few years back [1] so we should have the C++ implementation ready first. Apologies for the confusion. In the meantime, I'll give the Java implementation a more thorough review when I

Re: [C++] Coredump when writing Arrow table to Parquet file

2021-08-22 Thread Micah Kornfield
Hi Shuai, I responded on the JIRA (and have a PR out to fix the issue I think). -Micah On Fri, Aug 20, 2021 at 5:39 AM 张帅 wrote: > Hello everyone, > > I'm suffering from a coredump issue when writing Arrow table to Parquet > file. This issue is stable reproducible when compiling Arrow without >

Re: Flight SQL

2021-08-22 Thread Micah Kornfield
> > > To Micah's question, yes, we're going to hold a vote and treat it as > part of the spec, though even then like Flight itself it will be > "experimental" for a time. I'm not quite clear on what we would be voting on without the second implementation being present (even if it remains pre-1.0.

Re: [Rust] [DataFusion] Discuss new API for remote data sources

2021-08-22 Thread Andrew Lamb
There is now a design doc [1] available for comment [1] https://docs.google.com/document/d/1ZEZqvdohrot0ewtTNeaBtqczOIJ1Q0OnX9PqMMxpOF8/edit# On Thu, Aug 19, 2021 at 2:34 PM Andrew Lamb wrote: > I wanted to draw attention to a PR[1] in the DataFusion repository that > proposes a API designed