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

    https://github.com/apache/activemq-artemis/pull/2102#discussion_r190585655
  
    --- Diff: 
tests/compatibility-tests/src/main/resources/servers/artemisServer.groovy ---
    @@ -44,7 +44,7 @@ try {
         if (!type.startsWith("ARTEMIS-1")) {
             configuration.addAddressesSetting("#", new 
AddressSettings().setAutoCreateAddresses(true));
             if (globalMaxSize != null) {
    -            
configuration.getAddressesSettings().get("#").setPageSizeBytes(globalMaxSize).setAddressFullMessagePolicy(AddressFullMessagePolicy.PAGE)
    +            
configuration.getAddressesSettings().get("#").setPageSizeBytes(Long.parseLong(globalMaxSize)).setAddressFullMessagePolicy(AddressFullMessagePolicy.PAGE)
    --- End diff --
    
    I only added this fix because the PR build was failing.  I'm not sure how 
or where it broke.  It's not related to the previous commit.


---

Reply via email to