You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Alan Conway (JIRA)" <qp...@incubator.apache.org> on 2007/04/10 17:47:32 UTC

[jira] Created: (QPID-447) C++: Race in topic test

C++: Race in topic test
-----------------------

                 Key: QPID-447
                 URL: https://issues.apache.org/jira/browse/QPID-447
             Project: Qpid
          Issue Type: Test
            Reporter: Alan Conway


The topic test has a race condition. It's masked for now with a sleep in cpp/src/tests/topictest. 

If the publisher starts publishing before all the listeners are listening, then some listeners can miss some messages. This can cause a short topic test to hang, as some listeners may miss *all* the messages, including the report-request, and therefore never send a report.



-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (QPID-447) C++: Race in topic test

Posted by "Aidan Skinner (JIRA)" <qp...@incubator.apache.org>.
     [ https://issues.apache.org/jira/browse/QPID-447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Aidan Skinner updated QPID-447:
-------------------------------

          Component/s: C++ Client
    Affects Version/s: M3
        Fix Version/s: M3
             Assignee: Alan Conway

Changing component, assigning to reporter. Alan, is this still true?

> C++: Race in topic test
> -----------------------
>
>                 Key: QPID-447
>                 URL: https://issues.apache.org/jira/browse/QPID-447
>             Project: Qpid
>          Issue Type: Test
>          Components: C++ Client
>    Affects Versions: M3
>            Reporter: Alan Conway
>            Assignee: Alan Conway
>             Fix For: M3
>
>
> The topic test has a race condition. It's masked for now with a sleep in cpp/src/tests/topictest. 
> If the publisher starts publishing before all the listeners are listening, then some listeners can miss some messages. This can cause a short topic test to hang, as some listeners may miss *all* the messages, including the report-request, and therefore never send a report.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (QPID-447) C++: Race in topic test

Posted by "Gordon Sim (JIRA)" <qp...@incubator.apache.org>.
     [ https://issues.apache.org/jira/browse/QPID-447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gordon Sim resolved QPID-447.
-----------------------------

    Resolution: Fixed

Attached patch applied to trunk and qpid.0-10.

> C++: Race in topic test
> -----------------------
>
>                 Key: QPID-447
>                 URL: https://issues.apache.org/jira/browse/QPID-447
>             Project: Qpid
>          Issue Type: Test
>          Components: C++ Client
>    Affects Versions: M3
>            Reporter: Alan Conway
>            Assignee: Alan Conway
>            Priority: Minor
>             Fix For: M3
>
>         Attachments: topictest_statusqueue.patch
>
>
> The topic test has a race condition. It's masked for now with a sleep in cpp/src/tests/topictest. 
> If the publisher starts publishing before all the listeners are listening, then some listeners can miss some messages. This can cause a short topic test to hang, as some listeners may miss *all* the messages, including the report-request, and therefore never send a report.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (QPID-447) C++: Race in topic test

Posted by "Gordon Sim (JIRA)" <qp...@incubator.apache.org>.
     [ https://issues.apache.org/jira/browse/QPID-447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gordon Sim updated QPID-447:
----------------------------

    Priority: Minor  (was: Major)

Lower priority

> C++: Race in topic test
> -----------------------
>
>                 Key: QPID-447
>                 URL: https://issues.apache.org/jira/browse/QPID-447
>             Project: Qpid
>          Issue Type: Test
>          Components: C++ Client
>    Affects Versions: M3
>            Reporter: Alan Conway
>            Assignee: Alan Conway
>            Priority: Minor
>             Fix For: M3
>
>
> The topic test has a race condition. It's masked for now with a sleep in cpp/src/tests/topictest. 
> If the publisher starts publishing before all the listeners are listening, then some listeners can miss some messages. This can cause a short topic test to hang, as some listeners may miss *all* the messages, including the report-request, and therefore never send a report.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (QPID-447) C++: Race in topic test

Posted by "David Sommerseth (JIRA)" <qp...@incubator.apache.org>.
     [ https://issues.apache.org/jira/browse/QPID-447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Sommerseth updated QPID-447:
----------------------------------

    Attachment: topictest_statusqueue.patch

This patch gives an extra argument to topic_listener and topic_publisher, --status-queue.  This defines a queue where the publisher will check if all expected listeners are ready or not.  The publisher will wait until the right number of listeners have given a ready message.

The queue must be created before topic_publisher/topic_listener is run.  The queue must be deleted manually afterwards as well.

   qpid-config add queue topic_status
   topic_publisher --status-queue topic_status
   topic_listener --status-queue topic_status
   qpid-config del queue topic_status



> C++: Race in topic test
> -----------------------
>
>                 Key: QPID-447
>                 URL: https://issues.apache.org/jira/browse/QPID-447
>             Project: Qpid
>          Issue Type: Test
>          Components: C++ Client
>    Affects Versions: M3
>            Reporter: Alan Conway
>            Assignee: Alan Conway
>            Priority: Minor
>             Fix For: M3
>
>         Attachments: topictest_statusqueue.patch
>
>
> The topic test has a race condition. It's masked for now with a sleep in cpp/src/tests/topictest. 
> If the publisher starts publishing before all the listeners are listening, then some listeners can miss some messages. This can cause a short topic test to hang, as some listeners may miss *all* the messages, including the report-request, and therefore never send a report.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.