pitrou commented on a change in pull request #7733:
URL: https://github.com/apache/arrow/pull/7733#discussion_r453833318
##########
File path: cpp/src/arrow/array/concatenate.cc
##########
@@ -151,6 +151,10 @@ static Status PutOffsets(const std::shared_ptr<Buffer>&
src, Offset first_offset
return Status::Invalid("offset overflow while concatenating arrays");
}
+ // Concatenate can be called during IPC reads to append delta dictionaries,
+ // on non-validate input. Be sure to avoid reading out of buffer boundaries.
+ // XXX
Review comment:
Ha, I fixed that blunder already :-)
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
