You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/02/10 10:37:12 UTC

[GitHub] [arrow] jorisvandenbossche commented on pull request #11763: ARROW-14153: [C++][Dataset] Add support for batch_size in the ORC Scanner

jorisvandenbossche commented on pull request #11763:
URL: https://github.com/apache/arrow/pull/11763#issuecomment-1034757918


   > @pitrou
   All CI tests passed.
   
   CI is still failing here on the PR, but that seems to be CMake errors with AWS or LLVM. That doesn't seem to be directly related, but you might want to update your branch with the latest master to see if that resolves it.
   
   Additionally, there are some other comments that are not yet addressed I think:
   
   - Using `Result` instead of `Status` for the return value (https://github.com/apache/arrow/pull/11763/files#r776321965)


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org