Re: Review request for ARROW-7808's PR (Dataset Java API)

2021-01-29 Thread Micah Kornfield
I did another review mostly focused on the new allocation code. Thank you, I think this is close to mergeable once comments are addressed. If other Java committers/contributors want to have a look, please do so soon. On Thu, Jan 28, 2021 at 10:14 PM Hongze Zhang wrote: > Hi All, > > > Sorry to

Review request for ARROW-7808's PR (Dataset Java API)

2021-01-28 Thread Hongze Zhang
Hi All, Sorry to send a request to all but just would like to ask if anyone could be able to help finish the review for PR#7030[1]. As of now the PR contains following parts: 1. Base dataset API for Java language (which follows the shape of C++ API) 2. A JNI-based implementation of FileSyste