niyue commented on a change in pull request #11486:
URL: https://github.com/apache/arrow/pull/11486#discussion_r741521123
##########
File path: cpp/src/arrow/ipc/message.h
##########
@@ -441,6 +441,10 @@ class ARROW_EXPORT MessageReader {
virtual Result<std::unique_ptr<Message>> ReadNextMessage() = 0;
};
+// the first parameter of the function should be a pointer to metadata (aka.
+// org::apache::arrow::flatbuf::RecordBatch*)
+using FieldsLoaderFunction = std::function<Status(const void*,
io::RandomAccessFile*)>;
Review comment:
Hmmm, somehow this error is not reported when I compiled/ran the tests
locally. I got it fixed in the latest commit.
--
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]
