Github user RaiSaurabh commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2089#discussion_r188589506
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/qourum/SharedNothingBackupQuorum.java
 ---
    @@ -90,6 +92,7 @@ public SharedNothingBackupQuorum(StorageManager 
storageManager,
           this.networkHealthCheck = networkHealthCheck;
           this.voteRetries = voteRetries;
           this.voteRetryWait = voteRetryWait;
    +      this.quorumVoteWait = quorumVoteWait;
    --- End diff --
    
    The parameter is already validated when it is picked form the broker .xml.  
You can check class "FileConfigurationParser" L1322 and L1352 for validation of 
number greater than zero.


---

Reply via email to