Skip to content

[df] Enable ROOT thread safety on the distributed client#22848

Open
guitargeek wants to merge 1 commit into
root-project:masterfrom
guitargeek:distrdf
Open

[df] Enable ROOT thread safety on the distributed client#22848
guitargeek wants to merge 1 commit into
root-project:masterfrom
guitargeek:distrdf

Conversation

@guitargeek

Copy link
Copy Markdown
Contributor

Fixes a sporadic crash of the
pyunittests-roottest-python-distrdf-backends-all test (reported by CTest as "Subprocess aborted", i.e. the main pytest process dies from a fatal signal, most often surfacing at TestInitialization::test_initialization_method[dask]).

The problem was that the DistRDF client (the user's main process) drives ROOT from more than one thread without ROOT thread safety being enabled.

Fixes a sporadic crash of the
`pyunittests-roottest-python-distrdf-backends-all`
test (reported by CTest as "Subprocess aborted", i.e. the main pytest
process dies from a fatal signal, most often surfacing at
`TestInitialization::test_initialization_method[dask]`).

The problem was that the DistRDF client (the user's main process) drives
ROOT from more than one thread without ROOT thread safety being enabled.
@github-actions

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 19h 26m 17s ⏱️
 3 874 tests  3 874 ✅ 0 💤 0 ❌
78 808 runs  78 808 ✅ 0 💤 0 ❌

Results for commit 9664fa9.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant