Qpid
  1. Qpid
  2. QPID-3043

If a new session is created during failover, but before 'resume' is completed, the new session gets reattached again.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.6, 0.7, 0.8
    • Fix Version/s: 0.10
    • Component/s: None
    • Labels:
      None

      Description

      This bug was discovered while investigating QPID-2994

      There exists a race condition, where if a session is created (after the connection is setup and is marked OPEN) but before the resume method (in Connection.java) is called, it results in the new session being reattached again. This could result in unnecessary duplication of messages.

        Activity

        Hide
        Andrew Kennedy added a comment -

        The " connectionLost" field is scoped as "private static", so it is shared among all connections on the same JVM. This will lead to undesirable behaviour if only one of those connections is lost, and should probably be instance local. instead.

        Show
        Andrew Kennedy added a comment - The " connectionLost" field is scoped as "private static", so it is shared among all connections on the same JVM. This will lead to undesirable behaviour if only one of those connections is lost, and should probably be instance local. instead.
        Hide
        Rajith Attapattu added a comment -

        Fixed the error. I had accidentally marked the connectionLost variable as static.

        Show
        Rajith Attapattu added a comment - Fixed the error. I had accidentally marked the connectionLost variable as static.
        Hide
        Rajith Attapattu added a comment -

        The commit has been reviewed and was tested manually as part of a larger issue.

        Show
        Rajith Attapattu added a comment - The commit has been reviewed and was tested manually as part of a larger issue.

          People

          • Assignee:
            Rajith Attapattu
            Reporter:
            Rajith Attapattu
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development