panbingkun commented on PR #49627: URL: https://github.com/apache/spark/pull/49627#issuecomment-2611441271
> `PushVariantIntoScanSuite` is failing due to the change - it only recognizes `VariantGet`, but not the replaced expression (replacing runtime-replaceable expression happens before the rule). Essentially, I don't really think this change is a good idea. The implementation of `VariantGet` is simplified only by a bit, which is fixed and not likely to change, but the optimizer rules will get more complex (and there may be more rules targeting `VariantGet` in the future). I have fixed it, and another benefit of it is that when the `path` is foldable, it will run in `ConstantFolding`, thereby avoiding `eval it` every time and improving performance. @cloud-fan -- 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]
