thisisnic commented on pull request #10269:
URL: https://github.com/apache/arrow/pull/10269#issuecomment-838628591
> 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](https://issues.apache.org/jira/browse/ARROW-11705)" and see if
the two skipped tests work now that you've implemented this fix?
Done both now, and fixed spacing in a load of other places as well.
--
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:
us...@infra.apache.org
