Qpid
  1. Qpid
  2. QPID-4559

qpid-config deletes queue which contains messages

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.18
    • Fix Version/s: 0.21
    • Component/s: Python Tools
    • Labels:
      None

      Description

      qpid-config tool deletes a queue even if it contains some messages. The "force" command line options are not honored.

      1. jira-4599.patch
        4 kB
        Ernest Allen

        Activity

        Ernest Allen created issue -
        Hide
        Ernest Allen added a comment -

        Does the if_empty and if_unused tests in the python code before the call to delete the queue is made.

        Show
        Ernest Allen added a comment - Does the if_empty and if_unused tests in the python code before the call to delete the queue is made.
        Ernest Allen made changes -
        Field Original Value New Value
        Attachment qpid-config-bz903126-raise2.patch [ 12567587 ]
        Ted Ross made changes -
        Assignee Ted Ross [ tedross ]
        Hide
        Ted Ross added a comment -

        For the upstream development, it would be better to not implement a temporary fix but to provide the broker-side updates to do the proper pre-delete checks on the queue.

        Show
        Ted Ross added a comment - For the upstream development, it would be better to not implement a temporary fix but to provide the broker-side updates to do the proper pre-delete checks on the queue.
        Hide
        Gordon Sim added a comment -

        I agree with Ted, this should be fixed by adding the preconditions to the delete method.

        Show
        Gordon Sim added a comment - I agree with Ted, this should be fixed by adding the preconditions to the delete method.
        Ernest Allen made changes -
        Attachment bz-903126.diff1 [ 12570794 ]
        Ernest Allen made changes -
        Attachment qpid-config-bz903126-raise2.patch [ 12567587 ]
        Ernest Allen made changes -
        Attachment bz-903126.diff1 [ 12570794 ]
        Ernest Allen made changes -
        Comment [ Patches broker to check for flags ]
        Hide
        Ernest Allen added a comment - - edited

        The jira-4599.patch contains the changes to the C++ broker as well as the qpid-config python utility to support the if_empty and if_unused flags.
        It does not contain the temporary workaround in qpid-config that checks for the flags and raises the exception in the python code.

        Show
        Ernest Allen added a comment - - edited The jira-4599.patch contains the changes to the C++ broker as well as the qpid-config python utility to support the if_empty and if_unused flags. It does not contain the temporary workaround in qpid-config that checks for the flags and raises the exception in the python code.
        Ernest Allen made changes -
        Attachment jira-4599.patch [ 12573378 ]
        Ted Ross made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.21 [ 12323549 ]
        Resolution Fixed [ 1 ]
        Ernest Allen made changes -
        Affects Version/s 0.18 [ 12322451 ]
        Ernest Allen made changes -
        Fix Version/s 0.21 [ 12323549 ]
        Ernest Allen made changes -
        Fix Version/s 0.21 [ 12323549 ]
        Justin Ross made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        40d 3h 48m 1 Ted Ross 13/Mar/13 18:35
        Resolved Resolved Closed Closed
        138d 17m 1 Justin Ross 29/Jul/13 19:53

          People

          • Assignee:
            Ted Ross
            Reporter:
            Ernest Allen
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development