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);
                     });

Reply via email to