Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/18968
@viirya Isn't checkAnalysis supposed to catch such semantic errors ? In my
thinking, this particular error to make sure the left hand side number of args
matching the right hand side is to catch **user errors in the input query**.
After that is done either during analyzer or post analysis phase such as
checkAnalysis , we shouldn't be doing this particular check. My reason is that
, if optimizer causes a side effect such that it makes the original check
invalid, we shouldn't be returning the particular error that we return today as
that wouldn't mean much to the user as thats not the query he typed in ,
correct ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
