ianmcook commented on pull request #10269:
URL: https://github.com/apache/arrow/pull/10269#issuecomment-838481121
This looks pretty good to me! Just a few final things:
- Could you please ensure there are spaces added: `if(` → `if (` and `){`→
`) {`
- Could you search all the tests for "ARROW-11705" and see if the two
skipped tests work now that you've implemented this fix?
--
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]
