fresh-borzoni opened a new pull request, #353:
URL: https://github.com/apache/fluss-rust/pull/353
## Summary
ScanRecord in C++ held a raw pointer to the underlying scan data, making it
a borrowed view that dangled when ScanRecords was destroyed. Users couldn't
store or accumulate records across Poll() calls, unlike Rust (Arc<RecordBatch>)
where records are self-contained.
Fixed by wrapping the opaque ScanResultInner in shared_ptr with a custom
deleter that calls rust::Box::from_raw(). ScanRecord is now a value type:
copyable, storable, safe to accumulate. Zero-copy preserved (same FFI pointer
path), no Rust-side changes needed.
Updated docs to replace borrowing warnings with value-type semantics and
added an accumulation example to log-tables guide.
--
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]
