Github user tdas commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21700#discussion_r202922272
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/streaming/state/StateStoreSuite.scala
 ---
    @@ -64,21 +64,122 @@ class StateStoreSuite extends 
StateStoreSuiteBase[HDFSBackedStateStoreProvider]
         require(!StateStore.isMaintenanceRunning)
       }
     
    +  def updateVersionTo(provider: StateStoreProvider, currentVersion: => Int,
    +                      targetVersion: Int): Int = {
    --- End diff --
    
    Also, since you are frequently incrementing the version by 1 (i.e. 
targetVersion = currentVersion + 1, always), you can add another convenience 
method called `incrementVersion(provider, currentVersion)`



---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to