Skip to content

DPL: migrate away from MessageSet#15227

Closed
ktf wants to merge 13 commits intoAliceO2Group:devfrom
ktf:pr15227
Closed

DPL: migrate away from MessageSet#15227
ktf wants to merge 13 commits intoAliceO2Group:devfrom
ktf:pr15227

Conversation

@ktf
Copy link
Member

@ktf ktf commented Mar 26, 2026

Use a vector of messages instead. To be replaced by a B-Tree which
is able to keep track of all inputs / slots in a less rigid manner.

ktf added 13 commits March 26, 2026 13:57
Rest of the usecases removed. Abstract header / payload retrieval, with the idea that get_header / get_payload
will work on any range of fair::mq::MessagePtrs.
Rest of the usecases removed. Abstract header / payload retrieval, with the idea that get_header / get_payload
will work on any range of fair::mq::MessagePtrs.
Everything calculated on the fly
Use a vector of messages instead. To be replaced by a B-Tree which
is able to keep track of all inputs / slots in a less rigid manner.
@ktf ktf requested a review from a team as a code owner March 26, 2026 15:58
@github-actions
Copy link
Contributor

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@ktf
Copy link
Member Author

ktf commented Mar 26, 2026

I will most likely merge this in steps, however by running the DataRelayer benchmark getting rid of the MessageSet almost doubles performance.

@ktf ktf closed this Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant