danny0405 commented on pull request #2238: URL: https://github.com/apache/calcite/pull/2238#issuecomment-722199402
> 3457 is very related because it was the reason to disable fuzzer testing which is a significant test case for RexSimplify logic. > > For instance, you've just pushed `canNotMerge` which has comments like "if the semantics change". > How do you know you have test cases to cover the change? `RexProgramTest` already does that. Can we not make the fuzzer testing a random one ? It is hard to debug and figure out where is wrong. Although some stacktrace throws from 3457 code, that does not mean 3457's code is wrong. Each pr that changes the nullability can cause it fails. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
