anishshri-db commented on code in PR #52302:
URL: https://github.com/apache/spark/pull/52302#discussion_r2342418361


##########
sql/core/src/test/scala/org/apache/spark/sql/execution/python/streaming/TransformWithStateInPySparkStateServerSuite.scala:
##########
@@ -329,6 +329,7 @@ class TransformWithStateInPySparkStateServerSuite extends 
SparkFunSuite with Bef
     stateServer.handleListStateRequest(message)
     // Verify that the data is not read from Arrow stream. It is inlined.
     verify(transformWithStateInPySparkDeserializer, 
times(0)).readArrowBatches(any)
+    verify(transformWithStateInPySparkDeserializer).close()

Review Comment:
   We are basically only checking that `close` doesn't crash/assert here right ?



-- 
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: reviews-unsubscr...@spark.apache.org

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

Reply via email to