Uploaded image for project: 'Qpid'
  1. Qpid
  2. QPID-3026

The ApplicationRegistry object should be a singleton

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.9
    • Fix Version/s: 0.11
    • Component/s: Java Broker
    • Labels:
      None

      Description

      The ApplicationRegistry can currently have multiple instances, in order to cater for multiple brokers in the same VM. However, many parts of the code simply call ApplcationRegistry.getInstance() to obtain a reference, meaning multiple instances are ignored. The only time this is ever useful is for certain types of tests, therefore the capability to have multiple broker instances should be removed.

      Any tests that require multiple brokers can start a VM broker and an external, or two external brokers. In general, test-only code should not be made available to the rest of the application.

        Activity

        Hide
        k-wall Keith Wall added a comment -

        Patch to complete this improvement.

        Show
        k-wall Keith Wall added a comment - Patch to complete this improvement.
        Hide
        gemmellr Robbie Gemmell added a comment -

        Keith and I have been working on this together as discussed on the mailing list previously. It is being attached here to grant the appropriate permissions for inclusion.

        Show
        gemmellr Robbie Gemmell added a comment - Keith and I have been working on this together as discussed on the mailing list previously. It is being attached here to grant the appropriate permissions for inclusion.
        Hide
        gemmellr Robbie Gemmell added a comment -

        Closing out, patch applied.

        Show
        gemmellr Robbie Gemmell added a comment - Closing out, patch applied.

          People

          • Assignee:
            gemmellr Robbie Gemmell
            Reporter:
            andrew.kennedy Andrew Kennedy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development