vyasr commented on issue #47337:
URL: https://github.com/apache/arrow/issues/47337#issuecomment-3201974877

   > technically, it's not a C++ exception, it's just a process abort, so it's 
not possible to catch it
   
   Good point. I hadn't traced all the way down to what glog does in this case 
and I assumed Python was aborting due to an uncaught C++ exception calling 
terminate. I [now see that glog is aborting 
though](https://github.com/google/glog/blob/master/src/logging.cc#L1900). 
Perhaps it would make sense for arrow to use glog's 
[`InstallFailureFunction`](https://github.com/google/glog/blob/53d58e4531c7c90f71ddab503d915e027432447a/src/logging.cc#L1896)
 to instead throw an exception rather than aborting? Since Arrow uses 
exceptions in other places that seems like it would be more consistent.
   
   > nevertheless, we'd probably want to avoid aborting on invalid input like 
this, can you file an issue so that it can be fixed?
   
   Sure, I can do that.


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