Jefffrey commented on PR #10400: URL: https://github.com/apache/arrow-rs/pull/10400#issuecomment-5068916319
> > thinking about it some more, im not sure the tradeoff of introducing new From API implementations just for cleaning up some of the macro code is worth it 🤔 > > This `From` API is already implemented on `BinaryViewArray` and `StringArray` so it sounds a bit weird to me to not have it supported on `Binary`. But glad to drop this MR if you feel it's not useful. could you elaborate? i was referring to the `impl<OffsetSize: OffsetSizeTrait, const C: usize> From<Vec<Option<[u8; C]>>>` and co using const generics. these dont seem to exist on the other arrays? -- 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]
