[GitHub] [arrow] liyafan82 commented on pull request #9744: ARROW-12012: [Java][JDBC] Fix BinaryConsumer reallocation

2021-03-23 Thread GitBox
liyafan82 commented on pull request #9744: URL: https://github.com/apache/arrow/pull/9744#issuecomment-805416730 +1. Thanks @zxf -- 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 com

[GitHub] [arrow] liyafan82 commented on pull request #9744: ARROW-12012: [Java][JDBC] Fix BinaryConsumer reallocation

2021-03-22 Thread GitBox
liyafan82 commented on pull request #9744: URL: https://github.com/apache/arrow/pull/9744#issuecomment-804525245 Mostly look good. Thank you @zxf -- 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

[GitHub] [arrow] liyafan82 commented on pull request #9744: ARROW-12012: [Java][JDBC] Fix BinaryConsumer reallocation

2021-03-22 Thread GitBox
liyafan82 commented on pull request #9744: URL: https://github.com/apache/arrow/pull/9744#issuecomment-804525164 Rerunning integration tests. The failure seems unrelated. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [arrow] liyafan82 commented on pull request #9744: ARROW-12012: [Java][JDBC] Fix BinaryConsumer reallocation

2021-03-18 Thread GitBox
liyafan82 commented on pull request #9744: URL: https://github.com/apache/arrow/pull/9744#issuecomment-802541994 > @liyafan82 or @tianchen92 do you have time to review? @zxf Thanks for the patch. I will take a look. -- This is an automated message from the Apache Git Service. To re