[GitHub] [arrow] emkornfield commented on pull request #7908: ARROW-9358: [Integration] remove generated_large_batch.json

2020-08-14 Thread GitBox 


emkornfield commented on pull request #7908:
URL: https://github.com/apache/arrow/pull/7908#issuecomment-674346671


   Thanks.  +1 merging.



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] emkornfield commented on pull request #7908: ARROW-9358: [Integration] remove generated_large_batch.json

2020-08-13 Thread GitBox 


emkornfield commented on pull request #7908:
URL: https://github.com/apache/arrow/pull/7908#issuecomment-673877452


   @lidavidm this looks fine to me, but based on PR description I would have 
expected java to have some sort of file read removed?  Maybe its late but i 
just see added generation data which looks fine.



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] emkornfield commented on pull request #7908: ARROW-9358: [Integration] remove generated_large_batch.json

2020-08-11 Thread GitBox 


emkornfield commented on pull request #7908:
URL: https://github.com/apache/arrow/pull/7908#issuecomment-671835723


   Is there an urgency for this?  Have my hands full at work today and tomorrow 
at least, i can try to look later in the week?



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