nevi-me commented on pull request #8280: URL: https://github.com/apache/arrow/pull/8280#issuecomment-701050053
Hi @velvia, I pulled these changes from UL's fork while I was rebasing the changes from #6770. I only refactored the existing kernel functions there, but didn't expand any supported data types. The behaviour does change from what you had on your fork (https://github.com/apache/arrow/pull/8280#discussion_r495433674), so perhaps you can comment on whether this is fine, or bear that in mind when updating your fork. Regarding an ETA, the PR's still under review, so once it's approved, we'll be able to merge it in. Yes, you can open JIRAs (https://issues.apache.org/jira/projects/ARROW) then work on top of this PR. I've spent some time looking at the UL fork, and I think there might be changes there that the wider community would benefit from if you upstream them a bit more frequently. I understand that sometimes we might take longer than is ideal to complete PR reviews and merge them; but that's often a function of the sporadic availability of capacity from the Rust developers. We've also been very reliant on 2 people on the Parquet side, but I've been spending more time on the codebase so I can start picking up PRs on Parquet. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
