[ 
https://issues.apache.org/jira/browse/ARROW-1808?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16259426#comment-16259426
 ] 

ASF GitHub Bot commented on ARROW-1808:
---------------------------------------

wesm opened a new pull request #1337: ARROW-1808: [C++] Make RecordBatch, Table 
virtual interfaces for column access
URL: https://github.com/apache/arrow/pull/1337
 
 
   While this will cause some minor API breakage in parquet-cpp and some other 
downstream users, this is reasonably long overdue. It will permit 
implementations of the RecordBatch or Table interface that do lazy IO / data 
loading or lazy materialization of columns.
   
   I will write a patch to fix up parquet-cpp, and will look to see if glib is 
easy to fix. There's no good way to go about merging this patch since a green 
build is not possible, so once we're happy with the patch, I can merge this 
patch and then work on getting a green build in parquet-cpp so we don't have a 
broken build there for too long

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


> [C++] Make RecordBatch interface virtual to permit record batches that 
> lazy-materialize columns
> -----------------------------------------------------------------------------------------------
>
>                 Key: ARROW-1808
>                 URL: https://issues.apache.org/jira/browse/ARROW-1808
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Wes McKinney
>            Assignee: Wes McKinney
>              Labels: pull-request-available
>             Fix For: 0.8.0
>
>
> This should be looked at soon to prevent having to define a different virtual 
> interface for record batches. There are places where we are using the record 
> batch constructor directly, and in some third party code (like MapD), so this 
> might be good to get done for 0.8.0



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to