Re: [PR] [SPARK-45245][CONNECT][TESTS][FOLLOW-UP] Remove unneeded Matchers trait in the test [spark]

2024-03-11 Thread via GitHub
dongjoon-hyun commented on PR #45459: URL: https://github.com/apache/spark/pull/45459#issuecomment-1989103412 Merged to master. Thank you all. -- 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

Re: [PR] [SPARK-45245][CONNECT][TESTS][FOLLOW-UP] Remove unneeded Matchers trait in the test [spark]

2024-03-11 Thread via GitHub
dongjoon-hyun closed pull request #45459: [SPARK-45245][CONNECT][TESTS][FOLLOW-UP] Remove unneeded Matchers trait in the test URL: https://github.com/apache/spark/pull/45459 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] [SPARK-45245][CONNECT][TESTS][FOLLOW-UP] Remove unneeded Matchers trait in the test [spark]

2024-03-11 Thread via GitHub
cloud-fan commented on code in PR #45459: URL: https://github.com/apache/spark/pull/45459#discussion_r1519406352 ## core/src/test/scala/org/apache/spark/api/python/PythonWorkerFactorySuite.scala: ## @@ -33,7 +33,7 @@ import org.apache.spark.SparkFunSuite import

[PR] [SPARK-45245][CONNECT][TESTS][FOLLOW-UP] Remove unneeded Matchers trait in the test [spark]

2024-03-11 Thread via GitHub
HyukjinKwon opened a new pull request, #45459: URL: https://github.com/apache/spark/pull/45459 ### What changes were proposed in this pull request? This PR is a followup of https://github.com/apache/spark/pull/43023 that addresses a post-review comment. ### Why are the changes