Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Temporarily disable parallel backend tests due to frequent timeouts #19292

Merged

Conversation

WojciechMazur
Copy link
Contributor

Compilation tests run with -Ybackend-parallelism:N flag seem to be stuck when compiling probably due to deadlock or infinite loop. See
https://github.com/lampepfl/dotty/actions/runs/7243042276/job/19729290631#step:9:2254 which failed to compile tests/pos/reference/auto-param-tupling.scala

To be restored when underlying issue would be fixed

@WojciechMazur
Copy link
Contributor Author

Note: The deadlock can be reproduced on faster machines, eg. Apple M1

@nicolasstucki nicolasstucki linked an issue Dec 18, 2023 that may be closed by this pull request
@nicolasstucki nicolasstucki merged commit 8a3fc7a into scala:main Dec 18, 2023
16 checks passed
@WojciechMazur WojciechMazur deleted the tests/disabled-parallel-backend-tests branch December 18, 2023 20:14
WojciechMazur added a commit that referenced this pull request Dec 20, 2023
…9298)

Replaces #19297 and fixes #19293 

The deadlocks are now fixed by introduction of
`PostProcessorFrontendAccess.Lazy[T]` container for which initialization
is synchronized with the frontend Context, while providing a thread-safe
access lacking in original solution.

It now also audits where the unrestricted access to context was used and
replaces these usages with safer access.
Reverts #19292
@Kordyjan Kordyjan added this to the 3.4.0 milestone Dec 20, 2023
odersky pushed a commit to dotty-staging/dotty that referenced this pull request Jan 6, 2024
…ala#19298)

Replaces scala#19297 and fixes scala#19293 

The deadlocks are now fixed by introduction of
`PostProcessorFrontendAccess.Lazy[T]` container for which initialization
is synchronized with the frontend Context, while providing a thread-safe
access lacking in original solution.

It now also audits where the unrestricted access to context was used and
replaces these usages with safer access.
Reverts scala#19292
WojciechMazur added a commit that referenced this pull request Jul 10, 2024
…9298)

Replaces #19297 and fixes #19293 

The deadlocks are now fixed by introduction of
`PostProcessorFrontendAccess.Lazy[T]` container for which initialization
is synchronized with the frontend Context, while providing a thread-safe
access lacking in original solution.

It now also audits where the unrestricted access to context was used and
replaces these usages with safer access.
Reverts #19292
[Cherry-picked 33bdaac][modified]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nightly Dotty workflow of 2023-12-18 failed
3 participants