zhztheplayer commented on code in PR #13249:
URL: https://github.com/apache/arrow/pull/13249#discussion_r970258558
##########
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:
How about opening a ticket with title like "Add Java API to get imported
buffer size"? Since ARROW-17657 indeed introduces breaking change.
--
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]
