Github user franz1981 commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2089#discussion_r187949125
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/ha/ReplicatedPolicy.java
---
@@ -61,6 +61,8 @@
private final NetworkHealthCheck networkHealthCheck;
+ private int quorumVoteWait =
ActiveMQDefaultConfiguration.getDefaultQuorumVoteWait();
--- End diff --
It would be better to just use the one provided by the constructor and
declare this final if is not meant to be modified anymore
---