liyafan82 commented on pull request #7100: URL: https://github.com/apache/arrow/pull/7100#issuecomment-624989706
@rymurr Thanks a lot for your response. I have some small questions: 1. It is reasonable to set arrow.vector.max_allocation_bytes to 1MB for unit tests (to avoid OOM), but I do not think it reasonable to do so for integration tests, as they require large amounts of memory. 2. Please make sure to run the integration tests on your local machine, and make sure they pass before merging this PR. 3. Maybe we need to describe the way to run the integration tests somewhere (if you agree), maybe in the JavaDoc? ---------------------------------------------------------------- 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
