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

        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.
        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?
        Hide
        Robbie Gemmell added a comment -

        Removing it seems reasonable to me.

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development