timarmstrong opened a new pull request #31956: URL: https://github.com/apache/spark/pull/31956
### What changes were proposed in this pull request? Removed the custom toString implementation of AtLeastNNoneNulls. ### Why are the changes needed? It shows up wrong in the explain plan. The name of the function is wrong and the actual value of the first argument is not shown. Both of these would make it easier to understand the plan. ``` (12) Filter Input [3]: [c1#2410L, c2#2419, c3#2422] Condition : AtLeastNNulls(n, c1#2410L) ``` ### Does this PR introduce _any_ user-facing change? Only the explain plan changes if this function is used. ### How was this patch tested? Added a simple unit test to make sure that the toString output is correct. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
