(arrow-skyhook) 03/18: ARROW-15984: [C++] Change RecordBatchReader API to use Result<>

[ra...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

raulcd pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-skyhook.git

commit d196efa8b35274fb094f75974aae8068e5afb90d
Author: Tobias Zagorni <to...@zagorni.eu>
AuthorDate: Thu Mar 24 13:54:44 2022 +0100

    ARROW-15984: [C++] Change RecordBatchReader API to use Result<>
    
    Closes #12678 from zagto/recordbatchreader-result
    
    Authored-by: Tobias Zagorni <to...@zagorni.eu>
    Signed-off-by: Antoine Pitrou <an...@python.org>
---
 protocol/skyhook_protocol.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/protocol/skyhook_protocol.cc b/protocol/skyhook_protocol.cc
index c261048..f83b124 100644
--- a/protocol/skyhook_protocol.cc
+++ b/protocol/skyhook_protocol.cc
@@ -116,7 +116,7 @@ arrow::Status DeserializeTable(ceph::bufferlist& bl, bool use_threads,
   options.use_threads = use_threads;
   ARROW_ASSIGN_OR_RAISE(
       auto reader, arrow::ipc::RecordBatchStreamReader::Open(buffer_reader, options));
-  ARROW_RETURN_NOT_OK(reader->ReadAll(batches));
+  ARROW_ASSIGN_OR_RAISE(*batches, reader->ToRecordBatches());
   return arrow::Status::OK();
 }