pnoltes commented on PR #451:
URL: https://github.com/apache/celix/pull/451#issuecomment-1336516678

   This does not yet solve this issue.
   
   Looking at the logging:
   ```
   2022-12-04T19:21:58.0374265Z 52: 
   2022-12-04T19:21:58.0374884Z 52: Topic Senders:
   2022-12-04T19:21:58.0375433Z 52: |- Topic Sender default/test_invoke
   2022-12-04T19:21:58.0375747Z 52:    |- serializer type = json
   2022-12-04T19:21:58.0376160Z 52:    |- protocol type = envelope-v2
   2022-12-04T19:21:58.0376521Z 52:    |- url            = 
ipc:///tmp/pubsub-test-return (static)
   2022-12-04T19:21:58.0376753Z 52: 
   2022-12-04T19:21:58.0376903Z 52: 
   2022-12-04T19:21:58.0377061Z 52: Topic Receivers:
   2022-12-04T19:21:58.0377244Z 52: 
   2022-12-04T19:21:58.0377403Z 52: 
   2022-12-04T19:21:58.0377556Z 52: Topic Senders:
   2022-12-04T19:21:58.0377819Z 52: |- Topic Sender default/test_return
   2022-12-04T19:21:58.0378106Z 52:    |- serializer type = json
   2022-12-04T19:21:58.0378376Z 52:    |- protocol type = envelope-v2
   2022-12-04T19:21:58.0378721Z 52:    |- url            = 
ipc:///tmp/pubsub-test-invoke (static)
   2022-12-04T19:21:58.0378939Z 52: 
   2022-12-04T19:21:58.0379079Z 52: 
   2022-12-04T19:21:58.0379240Z 52: Topic Receivers:
   2022-12-04T19:21:58.0379511Z 52: |- Topic Receiver default/test_invoke
   2022-12-04T19:21:58.0379777Z 52:    |- serializer type = json
   2022-12-04T19:21:58.0380055Z 52:    |- protocol type = envelope-v2
   2022-12-04T19:21:58.0380402Z 52:    |- connected url   = 
ipc:///tmp/pubsub-test-return (static)
   2022-12-04T19:21:58.0380625Z 52: 
   2022-12-04T19:21:58.0381002Z 52: [2022-12-04T19:21:58] [   info] 
[celix_psa_zmq_v2] PSA_ZMQ_TP: Delaying first send for late joiners...
   2022-12-04T19:21:58.0381270Z 52: 
   2022-12-04T19:22:10.0391171Z 52: 
/home/runner/work/celix/celix/bundles/cxx_remote_services/integration/gtest/src/RemoteServicesIntegrationTestSuite.cc:151:
 Failure
   2022-12-04T19:22:10.0391842Z 52: Value of: promise.isSuccessfullyResolved()
   2022-12-04T19:22:10.0392185Z 52:   Actual: false
   2022-12-04T19:22:10.0392387Z 52: Expected: true
   ``` 
   
   I think there is a race condition in the pubsub topology manager / pubsub 
zmq, because the pubsub overview is missing a TopicReceiver to receive the 
promise return value. 
   
   For now I would like to merge this PR for its changes in CELIX_CXX and 
coverity issues and create a separate PR to tackle #397 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@celix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to