[GitHub] [calcite] jonathanswenson opened a new pull request, #2810: [CALCITE-2040] Arrow adapter (continued)

[GitBox <gi...@apache.org>]

jonathanswenson opened a new pull request, #2810:
URL: https://github.com/apache/calcite/pull/2810

   Builds on the work done by #2133 and additional commits by @julianhyde on his branch at https://github.com/julianhyde/calcite/tree/2040-arrow
   
   Added contributions: 
   + upgrade to arrow 8.0.0 (latest) allowing removal of the checked in jar - 7.0.0 likely will fit the same requirements, but I have not tried it.
   + Integrate the ArrowData class directly into the tests rather than depending on a gradle task to generate the arrow test file.
   + Fix a bug in early termination of the ArrowEnumerable if a filter does not match a given record batch. 
   + Begin implementing the tests that @julianhyde has specified. 
   
   There are a number of additional tests that have been called out. But many of them are currently not supported by the current approach. This appears to match the high level approach implemented by the cassandra adapter (hence the lack of support for several features). 


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@calcite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org