Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/7348#issuecomment-120673142
I've started writing a very basic fuzz-testing suite to check that
expression code generation does not fail if an expression is initialized with
NullType-d expressions as children and passes type checks after type coercion
is applied: https://github.com/apache/spark/compare/master...JoshRosen:fuzz-test
Running that suite against this patch shows that it fixes 9 of my test's
failures. I wouldn't read too much into this for now, since my suite needs
some work to be more useful, but just thought I'd share since it's a neat
testing technique and uncovered a few bugs that I had previously found via
other methods (such as `sum(null)` with a null literal).
---
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]
