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

Fix for "java.lang.IllegalStateException: Could not acquire lock(s)" in MavenLemminxWorkspaceReader #426

Conversation

vrubezhny
Copy link
Contributor

@vrubezhny vrubezhny commented Jun 24, 2023

Change Workspacereader populate thread to not use ProjectBuilder.build() API sue to the possibility
to be blocked on locking.

Also the PR #425 is used as the source of ProjectBuildingRequest locking configuration:

request.getUserProperties().setProperty("aether.syncContext.named.factory", "noop");

Thanks to @cstamas.

@vrubezhny vrubezhny mentioned this pull request Jun 24, 2023
@vrubezhny vrubezhny marked this pull request as draft June 24, 2023 13:30
@vrubezhny vrubezhny force-pushed the fixWorkspaceReader-populateWSRunner branch from 4edb4cb to 57c57a9 Compare June 26, 2023 22:26
@vrubezhny vrubezhny changed the title Change Workspace populate thread back to use 'getSnapshotProject' Fix for "java.lang.IllegalStateException: Could not acquire lock(s)" in MavenLemminxWorkspaceReader Jun 26, 2023
@vrubezhny vrubezhny marked this pull request as ready for review June 26, 2023 23:10
…in MavenLemminxWorkspaceReader

Change Workspacereader  populate thread to not use `ProjectBuilder.build()` API sue to the possibility
to be blocked on locking.

Also the PR eclipse-lemminx#425 is used as
the source of ProjectBuildingRequest locking configuration:
```
 request.getUserProperties().setProperty("aether.syncContext.named.factory", "noop");
```
Thanks to @cstamas.
@vrubezhny vrubezhny force-pushed the fixWorkspaceReader-populateWSRunner branch from 57c57a9 to 54f289d Compare June 27, 2023 00:32
@vrubezhny vrubezhny merged commit 4389085 into eclipse-lemminx:master Jun 27, 2023
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.

1 participant