Skip to content

DPL: allow non-owning TFileFileSystem#15179

Merged
ktf merged 2 commits intoAliceO2Group:devfrom
ktf:pr15179
Mar 17, 2026
Merged

DPL: allow non-owning TFileFileSystem#15179
ktf merged 2 commits intoAliceO2Group:devfrom
ktf:pr15179

Conversation

@ktf
Copy link
Member

@ktf ktf commented Mar 17, 2026

Given this filesystem is really a virtual entity, it makes
sense to allow for actual ownership of the TFile elsewhere.


Stack created with Sapling. Best reviewed with ReviewStack.

@ktf ktf requested a review from a team as a code owner March 17, 2026 13:56
@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 added 2 commits March 17, 2026 17:25
Given this filesystem is really a virtual entity, it makes
sense to allow for actual ownership of the TFile elsewhere.
@ktf ktf merged commit 87b5043 into AliceO2Group:dev Mar 17, 2026
15 of 17 checks passed
@ktf
Copy link
Member Author

ktf commented Mar 17, 2026

Compiles and trivial enough. Merging.

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