andygrove commented on PR #173: URL: https://github.com/apache/arrow-datafusion-python/pull/173#issuecomment-1425997994
> @andygrove just saw this. Yes. Question, what kind of error do we want to thrown when something happens like this you think? Should we add one for "UnImplemented"? Yes, in the core DataFusion crate we have `DataFusionError::NotImplemented(String)` so would be good to be consistent with 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org