westonpace commented on issue #6732: URL: https://github.com/apache/arrow-rs/issues/6732#issuecomment-2705112681
> Adding StructArray::try_new(fields, arrays, nulls, length) > Making StructArray::try_new(fields, arrays, nulls) return an error for empty fields instead of picking a default length I like these ideas and have made a proposal in #7247 . There is some discussion on also just changing the signature of `try_new` (instead of adding a new method) which would be more explicitly breaking but more likely to help others catch the error. Feel free to weigh in if you feel strongly about the approach. -- 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
