pnoltes opened a new pull request #353:
URL: https://github.com/apache/celix/pull/353
This PR fixes some issues in the pubsub interceptors and refactors how
interceptors are called from the pubsub admins.
The interceptor service is updated - backwards incompatible - for const
correctness
The TCP,ZMQ and websocket pubsub admin v2 are updated to that pre/post hooks
are only called once for a pubsub channel (topic and scope combination).
For the TCP pubsub admin v2 additional functionality (and complexity) is
added to that if a subscriber takes ownership of a message, a new message will
only be deserialized if there is an active interceptor. This is done to
maintain high performance use case for TCP pubsub.
The pubsub integration tests is refactored in to a integration folder and
additional tests for testing the (multiple) interceptors are added.
Removed interceptor support for the pubsub v1 admins.
The pubsub cmake alias targets used in the Celix project where updated to
that they match the cmake imported target names when Celix is installed.
Also several updates where added because of build problems:
- added -j $(nproc) argument for the Linux build to ensure not too much sub
make task are spawned. This is done because this giving the CI not enough
memory issues
- Updated the way C++17 and C++11 flags are done. This is done because the
OSX build stopped working (osx build machine update?)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
