ahshahid commented on PR #43854: URL: https://github.com/apache/spark/pull/43854#issuecomment-1834378548
@JoshRosen It may even be possible to minimize any hit on cache lookup too, but I think its not worthwhile as of now to attempt as it would make code a bit messy. The idea is that in this code , for a given plan, if it does not match, the child is also checked..The child check itself could be having 2 outcomes : No match of child, full match of child. so that info could be passed and depending on that the caller code can take steps to either use the child match or pass that info back to lookup function, which would skip the current node check and directly move to child. But that will alll be convoluted.. -- 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]
