Thanks everyone for the quick response! By the way I might raise this
review request a little bit late because I was working on some other
projects in the last few months either. Now I just have some time to
push this forward. :)


About ARROW-11776:

On Wed, 2021-08-04 at 08:45 -0700, Micah Kornfield wrote:
> One thing that can also help is if there is a way to divide any of 
> the PRs into smaller standalone components it would likely help get 
> them merged sooner

Yes. I used to refactor PR#10201 to have both fixes of ARROW-7272 (JNI
bridge of record batch) and of ARROW-11776 (Dataset write) in it and in
two different commits. Today I just make another individual PR for
ARROW-7272: https://github.com/apache/arrow/pull/10883 which I think
can be reviewed first. Hopefully it would ease the whole reviewing
process. :)


And regarding C data interface, I think I will be able to help
reviewing Roee's work which I think is already in a pretty good shape
now (maybe after a PR submitted). And once the codes get merged it
should be easy for Dataset to migrate from current data sharing
solution to the ABI cause I have extracted a utility layer for data
sharing in ARROW-7272's PR.

Thanks,
Hongze


Reply via email to