You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by yo...@apache.org on 2022/08/02 06:25:55 UTC

[bookkeeper] 02/22: Pre break loop when self create layoutZNode succeed. (#3335)

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

yong pushed a commit to branch branch-4.15
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git

commit 5557e7bba6c3bb060fb344d509dd3eccbe2e7a19
Author: Yan Zhao <ho...@apache.org>
AuthorDate: Wed Jul 27 19:09:07 2022 +0800

    Pre break loop when self create layoutZNode succeed. (#3335)
    
    Pre break loop when self create layoutZNode succeed.
    
    (cherry picked from commit ca7d7c23e3aa572befeb447157b3ef9d96bb1449)
---
 .../org/apache/bookkeeper/meta/ZkLedgerUnderreplicationManager.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/ZkLedgerUnderreplicationManager.java b/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/ZkLedgerUnderreplicationManager.java
index 9dcc81c162..c92d2d90d8 100644
--- a/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/ZkLedgerUnderreplicationManager.java
+++ b/bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/ZkLedgerUnderreplicationManager.java
@@ -188,9 +188,9 @@ public class ZkLedgerUnderreplicationManager implements LedgerUnderreplicationMa
                 try {
                     zkc.create(layoutZNode, builder.build().toString().getBytes(UTF_8),
                                zkAcls, CreateMode.PERSISTENT);
+                    break;
                 } catch (KeeperException.NodeExistsException nne) {
                     // someone else managed to create it
-                    continue;
                 }
             } else {
                 byte[] layoutData = zkc.getData(layoutZNode, false, null);