I'd be happy to PR a fix for JS today if someone can link me to Wes's PR.
On 5/21/19 11:02 AM, Wes McKinney wrote:
I agree also. As a practical use case, the results of a request made
with Arrow Flight might yield an empty result set. I'm not sure if
this needs to be formally noted in the specification documents but it
might not hurt.
If someone can fix the Java implementation we could enable the
integration test (minus JavaScript for now) in my PR
On Tue, May 21, 2019 at 12:47 AM Ravindra Pindikura <ravin...@dremio.com> wrote:
On Tue, May 21, 2019 at 10:35 AM Micah Kornfield <emkornfi...@gmail.com>
wrote:
Today, the format docs are ambiguous on whether zero sized batches are
supported. Wes opened a PR [1] for empty record batches that shows C++
handles them but Java and javascript fail to handle them.
I'd like to propose:
1. Make it explicit in the format docs, that 0 size record batches are
supported
2. Update Java and javascript implementations to work with them (I can put
the Java work on my backlog, but would need a volunteer for JS). And any
other implementations that don't currently handle them.
Thoughts?
Will need to add a test case for gandiva also - and fix if it shows up any
bugs. but, I agree we should support zero sized batches.
Thanks,
Micah
--
Thanks and regards,
Ravindra.
