[GitHub] [arrow] icexelloss commented on pull request #34645: GH-34644: [C++] Prefer unsafe casting by default in Substrait

["icexelloss (via GitHub)" <gi...@apache.org>]

icexelloss commented on PR #34645:
URL: https://github.com/apache/arrow/pull/34645#issuecomment-1477844785

   > (I am also not sure that just using unsafe instead of safe fully matches "user expectation". We might want to use a more finer grained CastOptions specification)
   
   I agree this doesn't not fully allow user to configure casting behavior - it just changes to a better default. Longer term we might want to think about how to expose the configuration. Currently there is no easy way to expose these via substrait so some work on the substrait spec side might be needed. Or some engine-wise configuration if we per-operation configuration is not needed.


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org