alex-balikov commented on a change in pull request #35816:
URL: https://github.com/apache/spark/pull/35816#discussion_r832657989



##########
File path: 
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStore.scala
##########
@@ -126,7 +126,10 @@ trait StateStore extends ReadStateStore {
 
   /**
    * Return an iterator containing all the key-value pairs in the StateStore. 
Implementations must
-   * ensure that updates (puts, removes) can be made while iterating over this 
iterator.
+   * ensure that updates (puts, removes) can be made while iterating over this 
iterator, but it is

Review comment:
       Split the ', but... ' part as a separate sentence. 
   
   /**
      * Return an iterator containing all the key-value pairs in the 
StateStore. Implementations must
      * ensure that updates (puts, removes) can be made while iterating over 
this iterator.
      * It is not required for implementations ...




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