[ 
https://issues.apache.org/jira/browse/ARTEMIS-4355?focusedWorklogId=871759&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-871759
 ]

ASF GitHub Bot logged work on ARTEMIS-4355:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 19/Jul/23 09:01
            Start Date: 19/Jul/23 09:01
    Worklog Time Spent: 10m 
      Work Description: amarkevich commented on code in PR #4545:
URL: https://github.com/apache/activemq-artemis/pull/4545#discussion_r1267776170


##########
artemis-quorum-ri/src/test/java/org/apache/activemq/artemis/quorum/zookeeper/CuratorDistributedLockTest.java:
##########
@@ -337,21 +333,8 @@ public void 
beNotifiedOfAlreadyUnavailableManagerAfterAddingListener() throws Ex
       }
    }
 
-   private boolean ensembleHasLeader() {
-      return 
testingServer.getServers().stream().filter(CuratorDistributedLockTest::isLeader).count()
 != 0;
-   }
-
-   private static boolean isLeader(TestingZooKeeperServer server) {
-      if (server.getInstanceSpecs().size() == 1) {
-         return true;
-      }
-      long leaderId = server.getQuorumPeer().getLeaderId();
-      long id = server.getQuorumPeer().getId();
-      return id == leaderId;
-   }
-
    private void stopMajorityNotLeaderNodes(boolean fromLast) throws Exception {
-      List<TestingZooKeeperServer> followers = 
testingServer.getServers().stream().filter(Predicates.not(CuratorDistributedLockTest::isLeader)).collect(Collectors.toList());
+      List<TestingZooKeeperServer> followers = testingServer.getServers();

Review Comment:
   Since leader api was removed there is no significant choice which instance 
stopped 





Issue Time Tracking
-------------------

    Worklog Id:     (was: 871759)
    Time Spent: 2h 10m  (was: 2h)

> Update Сurator to 5.5.0; Zookeeper 3.8.1
> ----------------------------------------
>
>                 Key: ARTEMIS-4355
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-4355
>             Project: ActiveMQ Artemis
>          Issue Type: Dependency upgrade
>          Components: clustering
>    Affects Versions: 2.29.0
>            Reporter: Alexey Markevich
>            Priority: Minor
>          Time Spent: 2h 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to