Qpid
  1. Qpid
  2. QPID-3698

org.apache.qpid.server.store.MessageStoreShutdownTest occasionally fails on Jenkins CI

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.15
    • Component/s: Java Broker, Java Tests
    • Labels:
      None

      Description

      https://builds.apache.org/view/M-R/view/Qpid/job/Qpid-Java-Java-TestMatrix/182/

      Stacktrace

      java.lang.NullPointerException
      at org.apache.qpid.server.protocol.InternalTestProtocolSession.getDelivers(InternalTestProtocolSession.java:102)
      at org.apache.qpid.server.store.MessageStoreShutdownTest.test(MessageStoreShutdownTest.java:65)
      at org.apache.qpid.test.utils.QpidTestCase.run(QpidTestCase.java:135)

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1d 1h 17m 1 Keith Wall 20/Dec/11 15:53
        Resolved Resolved Reopened Reopened
        11s 1 Keith Wall 20/Dec/11 15:53
        Reopened Reopened In Progress In Progress
        4s 1 Keith Wall 20/Dec/11 15:53
        In Progress In Progress Reviewable Reviewable
        3s 1 Keith Wall 20/Dec/11 15:53
        Reviewable Reviewable Resolved Resolved
        16d 20h 9m 1 Robbie Gemmell 06/Jan/12 12:02
        Resolved Resolved Closed Closed
        1132d 8h 3m 1 Rob Godfrey 11/Feb/15 20:06
        Rob Godfrey made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Robbie Gemmell made changes -
        Status Ready To Review [ 10006 ] Resolved [ 5 ]
        Fix Version/s 0.15 [ 12319043 ]
        Resolution Fixed [ 1 ]
        Hide
        Robbie Gemmell added a comment -

        Removing it seems reasonable to me.

        Show
        Robbie Gemmell added a comment - Removing it seems reasonable to me.
        Hide
        Keith Wall added a comment -

        Hi Robbie, would you mind reviewing this commit?

        Show
        Keith Wall added a comment - Hi Robbie, would you mind reviewing this commit?
        Keith Wall made changes -
        Assignee Keith Wall [ k-wall ] Robbie Gemmell [ gemmellr ]
        Keith Wall made changes -
        Status In Progress [ 3 ] Ready To Review [ 10006 ]
        Keith Wall made changes -
        Status Reopened [ 4 ] In Progress [ 3 ]
        Keith Wall made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Keith Wall made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Keith Wall [ k-wall ]
        Resolution Fixed [ 1 ]
        Hide
        Keith Wall added a comment -

        I think this is a problem with the test, which has been exposed by Rob's performance tuning work QPID-3687. The test relies on the action of the sub flush runner to deliver messages to the queue. There is an unlucky timeout where this delivery does not occur leading to the NPE.

        It seems to me that the test has little value and should be removed.

        Show
        Keith Wall added a comment - I think this is a problem with the test, which has been exposed by Rob's performance tuning work QPID-3687 . The test relies on the action of the sub flush runner to deliver messages to the queue. There is an unlucky timeout where this delivery does not occur leading to the NPE. It seems to me that the test has little value and should be removed.
        Keith Wall created issue -

          People

          • Assignee:
            Robbie Gemmell
            Reporter:
            Keith Wall
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development