Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21700#discussion_r202919419
--- 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 --
this is incorrect indenting by spark style guide. should be
```
def updateVersionTo(
provider: StateStoreProvider,
currentVersion: => Int,
targetVersion: Int): Int = {
```
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
