ActiveMQ
  1. ActiveMQ
  2. AMQ-3574

Enable <systemUsage> configuration in activemq.xml out of the box.

    Details

    • Patch Info:
      Patch Available

      Description

      Right now the <systemUsage> configuration in conf/activemq.xml is commented. If commented then the default systemUsage defined in BrokerService.getSystemUsage() takes effect.
      I don't think many users are aware of the default systemUsage settings. I know of some users that thought there are no systemUsage restrictions enforced when this configuration is commented.

      So rather than commenting out <systemUsage> in our default configuration, why not enabling it with the same settings as the default systemUsage (which anyway takes effect when the <systemUsage> config is commented)?
      IMHO it will make it more clear to users what limits are currently enforced.
      We should also leave a comment in the <systemUsage> config that disabling this config makes the broker use the default settings.

      1. AMQ-3574.patch
        2 kB
        Torsten Mielke

        Issue Links

          Activity

          Hide
          Torsten Mielke added a comment -

          Attaching a possible patch.

          Show
          Torsten Mielke added a comment - Attaching a possible patch.
          Hide
          Rob Davies added a comment -

          Fixed by SVN revision 1326948

          Show
          Rob Davies added a comment - Fixed by SVN revision 1326948
          Hide
          Rob Davies added a comment -

          changed memory limit to be the same as the default in org.apache.activemq.broker.BrokerService - 64mb by SVN revision 1327362

          Show
          Rob Davies added a comment - changed memory limit to be the same as the default in org.apache.activemq.broker.BrokerService - 64mb by SVN revision 1327362

            People

            • Assignee:
              Unassigned
              Reporter:
              Torsten Mielke
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development