cloud-fan commented on PR #48758: URL: https://github.com/apache/spark/pull/48758#issuecomment-2458713378
I agree with the problem here but I'm not quite sure if this is the right fix. This is not the first time that we are restricted by the trait-based tagging system. Extending a trait is static as it happens at compile but we really want to be more dynamic at runtime. For example, we added `Expression#stateful` and removed the trait `StatefulExpression` a while ago. I think the same thing happens to `NullIntelerant` now. Shall we also add `Expression#nullIntolerant`? Another example is `trait Nondeterministic`. We may hit the same issue in the future and try to remove it. -- 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]
