eliaperantoni opened a new issue, #14434:
URL: https://github.com/apache/datafusion/issues/14434

   ### Is your feature request related to a problem or challenge?
   
   For a query like:
   
   ```sql
   SELECT * FROM users WHERE password = NULL
   ```
   
   DataFusion does not emit any error. Of course this a valid query, but it's 
very likely that the user intended to do `password IS NULL` instead.
   
   We want to provide a rich message that _references and highlights locations 
in the original SQL query_, and contextualises and helps the user understand 
the pitfall. In the end, it would be possible to display a warning in a fashion 
akin to what was enabled by #13664 for some (fatal) errors:
   
   
![](https://github.com/user-attachments/assets/32efeb4f-aad9-41c9-a1e3-600a00d525c3)
   
   See #14429 for more information.
   
   ### Describe the solution you'd like
   
   Attach a well crafted `Diagnostic` to the `DataFusionError`, building on top 
of the foundations laid in #13664. See #14429 for more information.
   
   There would probably need to be some plumbing required to make DataFusion 
collect warnings without halting the overall execution. In this scenario, we 
want to use `Diagnostic::new_warning`.
   
   ### Describe alternatives you've considered
   
   _No response_
   
   ### Additional context
   
   _No response_


-- 
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.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

Reply via email to