Tushar7012 commented on issue #20031: URL: https://github.com/apache/datafusion/issues/20031#issuecomment-3830456846
Hi @Jefffrey , By “defensive error handling in AsyncFuncExpr::invoke_with_args()”, I’m not suggesting changing the core execution logic or adding broad try/catch-style behavior. What I had in mind is a small, targeted guard to handle cases where a nested async UDF invocation reaches an unexpected or unsupported state (which is what currently results in the internal error). Concretely, this would mean: Detecting when invoke_with_args() is reached in an invalid nested async context Failing fast with a clear, user-facing error message instead of hitting an internal error / panic Keeping this logic minimal and only scoped to async UDF nesting, not general execution paths That said, if you feel this guard is better placed elsewhere (or not needed once AsyncMapper::find_references() is fixed), I’m happy to drop or adjust this part and keep the change focused on the mapper + tests. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
