This is an automated email from the ASF dual-hosted git repository.

kfaraz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git


The following commit(s) were added to refs/heads/master by this push:
     new 7871e633c6 Fix bug in KillStalePendingSegments (#14961)
7871e633c6 is described below

commit 7871e633c68d54d1a7b925d000771a5ce7517de9
Author: Kashif Faraz <[email protected]>
AuthorDate: Mon Sep 11 15:18:15 2023 +0530

    Fix bug in KillStalePendingSegments (#14961)
---
 .../druid/server/coordinator/duty/KillStalePendingSegments.java       | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git 
a/server/src/main/java/org/apache/druid/server/coordinator/duty/KillStalePendingSegments.java
 
b/server/src/main/java/org/apache/druid/server/coordinator/duty/KillStalePendingSegments.java
index da730e20c0..ad07c3b317 100644
--- 
a/server/src/main/java/org/apache/druid/server/coordinator/duty/KillStalePendingSegments.java
+++ 
b/server/src/main/java/org/apache/druid/server/coordinator/duty/KillStalePendingSegments.java
@@ -118,9 +118,7 @@ public class KillStalePendingSegments implements 
CoordinatorDuty
     DateTime earliestActiveTaskStart = DateTimes.nowUtc();
     DateTime latestCompletedTaskStart = null;
     for (TaskStatusPlus status : statuses) {
-      if (status.getStatusCode() == null) {
-        // Unknown status
-      } else if (status.getStatusCode().isComplete()) {
+      if (status.getStatusCode() != null && 
status.getStatusCode().isComplete()) {
         latestCompletedTaskStart = DateTimes.laterOf(
             latestCompletedTaskStart,
             status.getCreatedTime()


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

Reply via email to