szehon-ho commented on PR #58412: URL: https://github.com/apache/spark/pull/58412#issuecomment-5483315916
Added the two follow-ups from cloud-fan's post-merge review of #58370: - Corrected the V1 runtime-filter Javadoc to describe dot-separated paths with each path part quoted as needed ([review comment](https://github.com/apache/spark/pull/58370#discussion_r3892275058)). - Resolved and cached `fullyPushedFilterAttributes()` from the primary `runtimeFilterAttrs` planning path, and updated the regression to exercise that entry point ([review comment](https://github.com/apache/spark/pull/58370#discussion_r3892275062)). The focused regression passes. -- 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]
