pitrou commented on code in PR #13249:
URL: https://github.com/apache/arrow/pull/13249#discussion_r968508989
##########
java/c/src/main/java/org/apache/arrow/c/ArrayImporter.java:
##########
@@ -141,9 +141,10 @@ private List<ArrowBuf> importBuffers(ArrowArray.Snapshot
snapshot) {
for (long bufferPtr : buffers) {
ArrowBuf buffer = null;
if (bufferPtr != NULL) {
- // TODO(roee88): an API for getting the size for each buffer is not yet
- // available
- buffer = new ArrowBuf(referenceManager, null, Integer.MAX_VALUE,
bufferPtr);
+ // See ARROW-17657: [C++] C data interface: Add field for byte length
of buffers in ArrowArray
Review Comment:
I'm not sure I understand this comment, is it a TODO?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
