alamb commented on issue #24701:
URL: https://github.com/apache/datafusion/issues/24701#issuecomment-5428054288

   I think this has a clear reproducer and would be a good first issue.
   
   The reproducer should be a SLT test
   
   then I would recommend trying to generate an error, probably in the 
`datafusion/datafusion/optimizer/src/common_subexpr_eliminate.rs` module, with 
a better message 
   
   The reason to put it in the same module (rather than in the sql planner for 
example) is 
   1. So it doesn't drift
   2. Doesn't trigger if a user is not using the common_subexpr_eliminate pass


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

Reply via email to