Details
-
Improvement
-
Status: Closed
-
Major
-
Resolution: Duplicate
-
None
-
None
Description
RecordBatch.schema() should not return &Arc<Schema>. It should either return &Schema or Arc<Schema>.
Given that the schema could be a large nested structure and other code may want to keep references to it, Arc<Schema> is probably best.
Returning a reference to an Arc just doesn't make much sense, IMHO, and I think I was responsible for introducing this early on.
Attachments
Issue Links
- is duplicated by
-
ARROW-8559 [Rust] Consolidate Record Batch reader traits in main arrow crate
-
- Resolved
-