This is an automated email from the ASF dual-hosted git repository.
aglinxinyuan pushed a commit to branch xinyuan-state-only
in repository https://gitbox.apache.org/repos/asf/texera.git
The following commit(s) were added to refs/heads/xinyuan-state-only by this
push:
new 94b874c53b fix fmt
94b874c53b is described below
commit 94b874c53b84391a60beea1b381d136735bd81ca
Author: Xinyuan Lin <[email protected]>
AuthorDate: Sun Apr 19 16:14:27 2026 -0700
fix fmt
---
.../scheduling/RegionExecutionCoordinator.scala | 19 ++++++++-----------
1 file changed, 8 insertions(+), 11 deletions(-)
diff --git
a/amber/src/main/scala/org/apache/texera/amber/engine/architecture/scheduling/RegionExecutionCoordinator.scala
b/amber/src/main/scala/org/apache/texera/amber/engine/architecture/scheduling/RegionExecutionCoordinator.scala
index 85c03081f6..889c9c94ee 100644
---
a/amber/src/main/scala/org/apache/texera/amber/engine/architecture/scheduling/RegionExecutionCoordinator.scala
+++
b/amber/src/main/scala/org/apache/texera/amber/engine/architecture/scheduling/RegionExecutionCoordinator.scala
@@ -182,8 +182,6 @@ class RegionExecutionCoordinator(
val actorRef = actorRefService.getActorRef(workerId)
// Remove the actorRef so that no other actors can find the
worker and send messages.
actorRefService.removeActorRef(workerId)
- asyncRPCClient.inputGateway.removeControlChannel(workerId)
- asyncRPCClient.outputGateway.removeControlChannel(workerId)
gracefulStop(actorRef, ScalaDuration(5,
TimeUnit.SECONDS)).asTwitter()
}
}.toSeq
@@ -212,15 +210,14 @@ class RegionExecutionCoordinator(
regionExecution: RegionExecution,
attempt: Int = 1
): Future[Unit] = {
- terminateWorkers(regionExecution).rescue {
- case err =>
- logger.warn(
- s"Failed to terminate region ${region.id.id} on attempt $attempt.
Retrying in ${killRetryDelay.inMilliseconds} ms.",
- err
- )
- Future
- .sleep(killRetryDelay)(killRetryTimer)
- .flatMap(_ => terminateWorkersWithRetry(regionExecution, attempt +
1))
+ terminateWorkers(regionExecution).rescue { case err =>
+ logger.warn(
+ s"Failed to terminate region ${region.id.id} on attempt $attempt.
Retrying in ${killRetryDelay.inMilliseconds} ms.",
+ err
+ )
+ Future
+ .sleep(killRetryDelay)(killRetryTimer)
+ .flatMap(_ => terminateWorkersWithRetry(regionExecution, attempt + 1))
}
}