This is an automated email from the ASF dual-hosted git repository.
cegerton pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git
The following commit(s) were added to refs/heads/trunk by this push:
new 4608070db15 MINOR: Fix error check in Connect Worker zombie fencing
(#13392)
4608070db15 is described below
commit 4608070db1546e9a0e3f5c544370b17d4e5bc4f8
Author: Yash Mayya <[email protected]>
AuthorDate: Tue Mar 14 21:39:52 2023 +0530
MINOR: Fix error check in Connect Worker zombie fencing (#13392)
---
.../runtime/src/main/java/org/apache/kafka/connect/runtime/Worker.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/connect/runtime/src/main/java/org/apache/kafka/connect/runtime/Worker.java
b/connect/runtime/src/main/java/org/apache/kafka/connect/runtime/Worker.java
index 8f9f727e25e..03d6404f748 100644
--- a/connect/runtime/src/main/java/org/apache/kafka/connect/runtime/Worker.java
+++ b/connect/runtime/src/main/java/org/apache/kafka/connect/runtime/Worker.java
@@ -709,7 +709,7 @@ public class Worker {
FenceProducersOptions fencingOptions = new
FenceProducersOptions()
.timeoutMs((int)
ConnectResource.DEFAULT_REST_REQUEST_TIMEOUT_MS);
return admin.fenceProducers(transactionalIds,
fencingOptions).all().whenComplete((ignored, error) -> {
- if (error != null)
+ if (error == null)
log.debug("Finished fencing out {} task producers
for source connector {}", numTasks, connName);
Utils.closeQuietly(admin, "Zombie fencing admin for
connector " + connName);
});