[GitHub] [arrow] rymurr commented on pull request #7347: ARROW-8230: [Java] Remove netty dependency from arrow-memory

2020-07-01 Thread GitBox


rymurr commented on pull request #7347:
URL: https://github.com/apache/arrow/pull/7347#issuecomment-652291140


   Thanks a lot @liyafan82 I have addressed your suggestions and rebased



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




[GitHub] [arrow] rymurr commented on pull request #7347: ARROW-8230: [Java] Remove netty dependency from arrow-memory

2020-06-29 Thread GitBox


rymurr commented on pull request #7347:
URL: https://github.com/apache/arrow/pull/7347#issuecomment-651081660


   looks like an ongoing github incident is causing build failures. Will rebase 
and rebuild once Github is back to normal



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




[GitHub] [arrow] rymurr commented on pull request #7347: ARROW-8230: [Java] Remove netty dependency from arrow-memory

2020-06-29 Thread GitBox


rymurr commented on pull request #7347:
URL: https://github.com/apache/arrow/pull/7347#issuecomment-651078278


   Thanks @liyafan82 I have updated based on your comments and the integration 
tests have passed.



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




[GitHub] [arrow] rymurr commented on pull request #7347: ARROW-8230: [Java] Remove netty dependency from arrow-memory

2020-06-25 Thread GitBox


rymurr commented on pull request #7347:
URL: https://github.com/apache/arrow/pull/7347#issuecomment-649372476


   Thanks for the reviews @liyafan82 I have updated based on your comments. 
Please let me know what you think!



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