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

    https://github.com/apache/activemq-artemis/pull/2102#discussion_r190442460
  
    --- 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 --
    
    should you merge this commit with the former one?


---

Reply via email to