asfimport opened a new issue, #270:
URL: https://github.com/apache/arrow-java/issues/270

   Per discussion 
<https://github.com/apache/arrow/pull/6133#discussion_r364906302>.
   
   The fact that we have reader/writer settings in `getFieldBuffers` is wrong. 
To clarify, `getFieldBuffers` is distinct from `getBuffers`. The former should 
be for getting access to underlying data for higher-performance algorithms. The 
latter is for sending the data over the wire. Seems we've mixed up use of both.
   
    
   
   Currently in `VectorUnloader`, we used `getFieldBuffers` to create 
`ArrowRecordBatch` that’s why we keep writer/reader indices in 
`getFieldBuffers`, we should use `getBuffers` instead.
   
   **Reporter**: [Ji Liu](https://issues.apache.org/jira/browse/ARROW-7539) / 
@tianchen92
   #### PRs and other links:
   - [GitHub Pull Request 
apache/arrow#6156](https://github.com/apache/arrow/pull/6156)
   
   <sub>**Note**: *This issue was originally created as 
[ARROW-7539](https://issues.apache.org/jira/browse/ARROW-7539). Please see the 
[migration documentation](https://github.com/apache/arrow/issues/14542) for 
further details.*</sub>


-- 
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: issues-unsubscr...@arrow.apache.org.apache.org

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

Reply via email to