activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From michaelandrepearce <...@git.apache.org>
Subject [GitHub] activemq-artemis issue #1268: ARTEMIS-1162: Make new TimedBuffer Configurabl...
Date Sat, 13 May 2017 08:03:52 GMT
Github user michaelandrepearce commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1268
  
    I actually started with a boolean flag option, but then thought about the other bits being
added and before we know it we'd have quite a few flags which equate to lots of config options
to be added in broker.xml
    
    Where as having an enum from a user perspective means you simply define which timed buffer
type you want, and is a little more plain English with what you're selecting.
    
    Maybe I can see if I can make the enum for config so we don't explode the config paeans
but re work the reintroduction of the old behaviour like you propose. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message