jayzhan211 commented on PR #11999: URL: https://github.com/apache/datafusion/pull/11999#issuecomment-2291402746
> I personally think the internal error makes more sense but I don't have a problem changing this I think the idea about returning internal error was that if these errors get hit, then it signals some bug in the datafusion code I think the problem is typically for a function that is both used directly from down stream project and internally. We return internal error even it is used incorrectly from down stream project, which is actually not the bug from datafusion. #11992 is one of the case -- 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...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
