Skip to content

Commit

Permalink
R client: fix warning caused by not checking return value of function (
Browse files Browse the repository at this point in the history
  • Loading branch information
kosak authored Apr 18, 2024
1 parent d02640d commit d3e932a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion R/rdeephaven/src/client.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -545,7 +545,7 @@ class TableHandleWrapper {

std::shared_ptr<arrow::RecordBatchReader> record_batch_reader = arrow::RecordBatchReader::Make(empty_record_batches).ValueOrDie();
ArrowArrayStream* stream_ptr = new ArrowArrayStream();
arrow::ExportRecordBatchReader(record_batch_reader, stream_ptr);
deephaven::client::utility::OkOrThrow(DEEPHAVEN_LOCATION_EXPR(arrow::ExportRecordBatchReader(record_batch_reader, stream_ptr)));

// XPtr is needed here to ensure Rcpp can properly handle type casting, as it does not like raw pointers
return Rcpp::XPtr<ArrowArrayStream>(stream_ptr, true);
Expand Down

0 comments on commit d3e932a

Please sign in to comment.