HeartSaVioR commented on code in PR #55468:
URL: https://github.com/apache/spark/pull/55468#discussion_r3242308927


##########
sql/core/src/test/scala/org/apache/spark/sql/execution/streaming/state/RocksDBStateStoreSuite.scala:
##########
@@ -1856,6 +1856,53 @@ class RocksDBStateStoreSuite extends 
StateStoreSuiteBase[RocksDBStateStoreProvid
     }
   }
 
+  test("SPARK-56539: validate state row format in prefixScan") {

Review Comment:
   This test and below do not really check whether we do validate state row 
format or not - if it's technically hard to create a test verifying what we 
just added, don't hesitate to call it out.



-- 
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]

Reply via email to