jp0317 commented on PR #6958:
URL: https://github.com/apache/arrow-rs/pull/6958#issuecomment-2580972469

   > I wonder if there is some way to work this comment into this [#6737 
(comment)](https://github.com/apache/arrow-rs/issues/6737#issuecomment-2551572811)
   > 
   > I think it articulates something very important, that simply blindly 
changing panics to results is not desirable
   
   Thanks for the comment! Added a note to emphasize that it's about reporting 
invalidity of user input as error results which does not necessarily replaces 
panic/assert for sanity checks, PTAL!


-- 
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]

Reply via email to