Qpid
  1. Qpid
  2. QPID-2815

Refactor broker startup to enable a uniform method of starting the broker

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.7
    • Fix Version/s: 0.13
    • Component/s: None
    • Labels:
      None

      Description

      Update the way the broker is started to decouple it from any particular protocol version or transport.

        Activity

        Hide
        Andrew Kennedy added a comment -

        Move forward to 0.9

        Show
        Andrew Kennedy added a comment - Move forward to 0.9
        Hide
        Keith Wall added a comment -

        This change presents a clean interface that can be used to start/stop the broker from within an existing application, regardless of protocol.

        This will be used by the test-framework (to rapidly start/stop the broker during test setup/tearDown), and could be used by other applications that require an embedded broker.

        Show
        Keith Wall added a comment - This change presents a clean interface that can be used to start/stop the broker from within an existing application, regardless of protocol. This will be used by the test-framework (to rapidly start/stop the broker during test setup/tearDown), and could be used by other applications that require an embedded broker.
        Hide
        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
        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
        Robbie Gemmell added a comment -

        Closing out, patch applied.

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

        Reopening to add a unit test for the vastly simplified Main class to ensure the command line argument parsing is functioning as expected.

        Show
        Robbie Gemmell added a comment - Reopening to add a unit test for the vastly simplified Main class to ensure the command line argument parsing is functioning as expected.
        Hide
        Robbie Gemmell added a comment -

        Hi Keith, could you review this latest change and close out the JIRA if you are happy?

        I added a unit test for Main and updated the handling of the logWatch argument slightly.

        Show
        Robbie Gemmell added a comment - Hi Keith, could you review this latest change and close out the JIRA if you are happy? I added a unit test for Main and updated the handling of the logWatch argument slightly.
        Hide
        Keith Wall added a comment -

        I've reviewed the additions and I have no comments.
        cheers Keith.

        Show
        Keith Wall added a comment - I've reviewed the additions and I have no comments. cheers Keith.

          People

          • Assignee:
            Keith Wall
            Reporter:
            Andrew Kennedy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development