tustvold commented on PR #7247:
URL: https://github.com/apache/arrow-rs/pull/7247#issuecomment-2705112983
> which suggested adding try_new_with_length (yet another alternative)
I'm personally less a fan of this, as it isn't materially different from the
current state of play - where there are multiple alternative methods.
I guess I was thinking something along the lines of
```
pub fn try_new(
len: usize,
fields: Fields,
arrays: Vec<ArrayRef>,
nulls: Option<NullBuffer>,
) -> Result<Self, ArrowError>
```
That being said, this would be potentially quite a disruptive change, and I
am not sure how common empty StructArray actually are, I can't help feeling
most of the time they'd arise from a deficiency in a projection system, rather
than someone actually creating them...
For an additional data point, RecordBatch::try_new is consistent with
StructArray
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
