s-maling-celonis commented on PR #58856: URL: https://github.com/apache/spark/pull/58856#issuecomment-6018081525
Thanks @HeartSaVioR, agreed, this isn't critical current RC's then. The single-pass analyzer is still internal and off by default (`spark.sql.analyzer.singlePassResolver.enabled` / `enabledTentatively` both default to false; dual-run only defaults on under Utils.isTesting). Today only users who opt into the experimental single-pass analyzer have injected hint rules silently skipped. It does become a blocker once singlePassResolver.enabled defaults to true, though. At that point every extension using injectHintResolutionRule would silently lose its rules, which breaks behavior that works today. So instead it can be merged to master and branch-4.x now?, and ship in 4.4.0? -- 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]
