[GitHub] activemq-artemis pull request #1853: ARTEMIS-1663 - Add new message count an...

[michaelandrepearce]

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

    https://github.com/apache/activemq-artemis/pull/1853#discussion_r166387776
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/codec/PageCountPendingImpl.java
 ---
    @@ -43,6 +44,8 @@ public PageCountPendingImpl(long queueID, long pageID, 
int inc) {
     
        long pageID;
     
    +   long size;
    --- End diff --
    
    Eg it’s typical for -1 to be used to represent null vs 0 as no size , 
slight semantic difference but important none the less


---