Skip to content

DPL: Fix memory leak in variant_helper#13051

Merged
ktf merged 1 commit into
AliceO2Group:devfrom
ktf:pr13051
Apr 22, 2024
Merged

DPL: Fix memory leak in variant_helper#13051
ktf merged 1 commit into
AliceO2Group:devfrom
ktf:pr13051

Conversation

@ktf

@ktf ktf commented Apr 19, 2024

Copy link
Copy Markdown
Member

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
Copy Markdown
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

ktf commented Apr 19, 2024

Copy link
Copy Markdown
Member Author

@benedikt-voelkel can we have a +all?

@benedikt-voelkel

Copy link
Copy Markdown
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
Copy Markdown
Contributor

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

@ktf

ktf commented Apr 22, 2024

Copy link
Copy Markdown
Member Author

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

ktf commented Apr 22, 2024

Copy link
Copy Markdown
Member Author

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