wgtmac commented on PR #39393: URL: https://github.com/apache/arrow/pull/39393#issuecomment-1879584886
Sorry for chiming in not sooner. I agree that it would be good to break down this PR into smaller ones. Otherwise, it would be challenging to get properly reviewed. To provide some context, this [doc](https://docs.google.com/document/d/1SeVcYudu6uD9rb9zRAnlLGgdauutaNZlAaS0gVzjkgM/edit#heading=h.3hx86uo8dp88) was originally inspired by offline discussion from @mapleFU @baibaichen @binmahone and me and further reviewed by @emkornfield @fatemehp. We have somewhat reached a consensus on the API and need more discussion on concrete PRs to refine the implementation. @huberylee I have seen your detailed comment and replied to some of them in that doc. It would be good to go through the doc before heading to the implementation. Otherwise it might waste time of you and people involved in the original discussion. I will take a look at this PR over the weekend and see if we can be on the same page quickly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
