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

    https://github.com/apache/activemq-artemis/pull/2490#discussion_r246383235
  
    --- Diff: 
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/wireformat/SessionCreateConsumerMessage.java
 ---
    @@ -113,6 +133,7 @@ public int hashCode() {
           result = prime * result + (browseOnly ? 1231 : 1237);
           result = prime * result + ((filterString == null) ? 0 : 
filterString.hashCode());
           result = prime * result + (int) (id ^ (id >>> 32));
    +      result = prime * result + priority;
    --- End diff --
    
    I would uses `Integer::hashCode(priority)` that would `agitate` the value 


---

Reply via email to