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

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

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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

        Hide
        k-wall 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
        k-wall 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.
        Hide
        k-wall Keith Wall added a comment -

        Hi Robbie, would you mind reviewing this commit?

        Show
        k-wall Keith Wall added a comment - Hi Robbie, would you mind reviewing this commit?
        Hide
        gemmellr Robbie Gemmell added a comment -

        Removing it seems reasonable to me.

        Show
        gemmellr Robbie Gemmell added a comment - Removing it seems reasonable to me.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development