Rich-T-kid commented on PR #10317: URL: https://github.com/apache/arrow-rs/pull/10317#issuecomment-5132454956
after this is merged ill make a PR to wire up call sites to use the new `try_*()` methods. Ideally we can propogate these errors up the call stack as https://docs.rs/arrow/latest/arrow/error/enum.ArrowError.html#variant.MemoryError > we could technically match on the errors and if we hit AllocationError just call handle_alloc_error as we essentially did before im not opposed to this but is there any reason why we shouldnt just pass up OOM errors if possible instead of panicking? -- 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]
