LuciferYang commented on PR #58424: URL: https://github.com/apache/spark/pull/58424#issuecomment-5548572978
Three small additions: a case with the InSubquery in the SELECT list or a CASE WHEN (the rule scans all expressions, so it fires there too); a direct test that re-applying the rule leaves the plan unchanged (it runs in two fixedPoint batches; idempotence is currently covered only indirectly by the structural tests); and a precondition assert in the correlated test that with the rewrite off the plan still holds an InSubquery with outer references, so a future pullup change cannot make this negative test vacuous. -- 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]
