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

[WIP] Improve handling for parallel Scala CLI runs #3399

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Gedochao
Copy link
Contributor

@Gedochao Gedochao commented Dec 31, 2024

@@ -37,8 +38,10 @@ object Bloop {
): Either[Throwable, Boolean] =
try {
logger.debug("Listing BSP build targets")
val results = buildServer.workspaceBuildTargets()
.get(buildTargetsTimeout.length, buildTargetsTimeout.unit)
val results = retry()(logger) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Och, this can also fail?

Copy link
Contributor Author

@Gedochao Gedochao Dec 31, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yup, thus the ExecutionException catch case 😞

@Gedochao Gedochao force-pushed the maintenance/fix-parallel-cli-runs branch from f593f60 to 5971959 Compare December 31, 2024 11:11
@Gedochao Gedochao changed the title Improve handling for parallel Scala CLI runs [WIP] Improve handling for parallel Scala CLI runs Dec 31, 2024
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.

ArrayIndexOutOfBoundsException when running more than one process in parallel
2 participants