changsun20 commented on PR #15696: URL: https://github.com/apache/datafusion/pull/15696#issuecomment-2802985874
@comphead I understand your concern. If displaying warnings to end users is what you'd like to see in this PR, could you confirm if @eliaperantoni's proposed solution in #14434 of "replacing Result with DatafusionResult" aligns with what you're thinking? My concern is that this approach might be too invasive, since we're dealing with warnings that should be passed to the end without interrupting execution, rather than immediate errors. However, as discussed in the issue, this could be more robust long-term. Please share your thoughts and preference. If this is the direction the community chooses, I'll convert this PR to draft status and implement that approach later. -- 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
