Skip to content

Conversation

@ktf
Copy link
Member

@ktf ktf commented Apr 19, 2024

DPL: Fix memory leak in variant_helper

@ktf ktf requested a review from a team as a code owner April 19, 2024 12:59
@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-apass3
async-2023-pbpb-apass4
async-2022-pp-apass6-2023-PbPb-apass2
async-2022-pp-apass4
async-2022-pp-apass4-accepted
async-2022-pp-apass6-2023-PbPb-apass2-accepted
async-2023-pbpb-apass3-accepted
async-2023-pbpb-apass4-accepted

@ktf
Copy link
Member Author

ktf commented Apr 19, 2024

@benedikt-voelkel can we have a +all?

@benedikt-voelkel
Copy link
Contributor

Sounds good!
I will update the workflow so that we will have some better usage by Monday, including an +async-label all

@martenole
Copy link
Contributor

Unfortunately the CCDB backend is still leaking memory at the same rate as before with this PR included

@ktf
Copy link
Member Author

ktf commented Apr 22, 2024

Mmmm... did you actually include the PR separately? For some reason is not yet merged, even if I though I did click on "merge".

@ktf ktf merged commit f5ebb69 into AliceO2Group:dev Apr 22, 2024
@ktf ktf deleted the pr13051 branch April 22, 2024 06:53
@ktf
Copy link
Member Author

ktf commented Apr 22, 2024

Now merged.

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.

3 participants