Skip to content

Commit f7d0e70

Browse files
committed
removing clera
1 parent 63573db commit f7d0e70

File tree

3 files changed

+5
-3
lines changed

3 files changed

+5
-3
lines changed

sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/RocksDBStateStoreProvider.scala

+1
Original file line numberDiff line numberDiff line change
@@ -488,6 +488,7 @@ private[sql] class RocksDBStateStoreProvider
488488
existingStore match {
489489
case Some(stateStore: RocksDBStateStore) =>
490490
// Reuse existing store for getWriteStore case
491+
StateStoreThreadLocalTracker.setUsedForWriteStore(true)
491492
stateStore
492493
case Some(_) =>
493494
throw new IllegalArgumentException("Existing store must be a RocksDBStateStore")

sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStoreRDD.scala

+4-1
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,13 @@ object StateStoreThreadLocalTracker {
3737
def setStore(store: ReadStateStore): Unit = readStore.set(store)
3838

3939
def getStore: Option[ReadStateStore] = {
40-
usedForWriteStore.set(true)
4140
Option(readStore.get())
4241
}
4342

43+
def setUsedForWriteStore(used: Boolean): Unit = {
44+
usedForWriteStore.set(used)
45+
}
46+
4447
def isUsedForWriteStore: Boolean = usedForWriteStore.get()
4548

4649
def clearStore(): Unit = readStore.remove()

sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/package.scala

-2
Original file line numberDiff line numberDiff line change
@@ -113,12 +113,10 @@ package object state {
113113
val ctxt = TaskContext.get()
114114
ctxt.addTaskCompletionListener[Unit](_ => {
115115
if (!StateStoreThreadLocalTracker.isUsedForWriteStore) store.release()
116-
StateStoreThreadLocalTracker.clearStore()
117116
})
118117
ctxt.addTaskFailureListener(new TaskFailureListener {
119118
override def onTaskFailure(context: TaskContext, error: Throwable): Unit =
120119
if (!StateStoreThreadLocalTracker.isUsedForWriteStore) store.abort()
121-
StateStoreThreadLocalTracker.clearStore()
122120
})
123121
cleanedF(store, iter)
124122
}

0 commit comments

Comments
 (0)