liyafan82 commented on pull request #7347:
URL: https://github.com/apache/arrow/pull/7347#issuecomment-652739238
> @liyafan82 if you aren't already please make sure you use the merge script
under dev to merge PRs
@emkornfield Thanks a lot for your kind reminder. I will use the
liyafan82 commented on pull request #7347:
URL: https://github.com/apache/arrow/pull/7347#issuecomment-652319908
> Thanks a lot @liyafan82 I have addressed your suggestions and rebased
@rymurr Thanks for your work. Will merge when it turns green.
liyafan82 commented on pull request #7347:
URL: https://github.com/apache/arrow/pull/7347#issuecomment-651474250
@rymurr Thanks for your work. A few typos.
I think it would be ready for merge.
This is an automated
liyafan82 commented on pull request #7347:
URL: https://github.com/apache/arrow/pull/7347#issuecomment-650739545
Mostly looks good to me. There are a few minor issues.
Since it involves some fundamental classes, could you please make sure our
integration tests pass? @rymurr
liyafan82 commented on pull request #7347:
URL: https://github.com/apache/arrow/pull/7347#issuecomment-650685726
@rymurr Thanks for your effort. I will make another pass today.
This is an automated message from the Apache
