Skip to content

Commit 6c778cd

Browse files
committed
remove unused code and irrelevant comment
1 parent 5d16f69 commit 6c778cd

File tree

2 files changed

+1
-16
lines changed

2 files changed

+1
-16
lines changed

sql/pipelines/src/main/scala/org/apache/spark/sql/pipelines/graph/PipelineExecution.scala

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@
1717

1818
package org.apache.spark.sql.pipelines.graph
1919

20-
import scala.annotation.unused
21-
2220
import org.apache.spark.sql.pipelines.logging.{
2321
ConstructPipelineEvent,
2422
EventLevel,
@@ -115,17 +113,4 @@ class PipelineExecution(context: PipelineUpdateContext) {
115113
)
116114
}
117115
}
118-
119-
@unused(
120-
"TODO(SPARK-51727): connect this API with spark connect backend and expose it to the user"
121-
)
122-
def stopPipeline(): Unit = synchronized {
123-
graphExecution
124-
.getOrElse(
125-
throw new IllegalStateException(
126-
"Pipeline execution has not started yet."
127-
)
128-
)
129-
.stop()
130-
}
131116
}

sql/pipelines/src/main/scala/org/apache/spark/sql/pipelines/graph/TriggeredGraphExecution.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ class TriggeredGraphExecution(
110110
val thread = buildTopologicalExecutionThread()
111111
UncaughtExceptionHandler.addHandler(
112112
thread, {
113-
case _: InterruptedException => // stop from UI leads to InterruptedException. Do nothing.
113+
case _: InterruptedException =>
114114
case _ =>
115115
try {
116116
stopInternal(stopTopologicalExecutionThread = false)

0 commit comments

Comments
 (0)