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

Avoid refreshing bundles whose dependency closure includes JDT-LS. #2951

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

snjeza
Copy link
Contributor

@snjeza snjeza commented Nov 9, 2023

@rgrunber rgrunber added this to the End November 2023 milestone Nov 9, 2023
Copy link
Contributor

@rgrunber rgrunber left a comment

Choose a reason for hiding this comment

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

Overall, this seems to fix the issue! Just wouldn't mind having a closer look.

I was reproducing this (using VS Code) by making sure vscode-pde was installed, and then applying the following patch to JDT-LS :

diff --git a/org.eclipse.jdt.ls.product/languageServer.product b/org.eclipse.jdt.ls.product/languageServer.product
index 3b97b4ff..4389cb7e 100644
--- a/org.eclipse.jdt.ls.product/languageServer.product
+++ b/org.eclipse.jdt.ls.product/languageServer.product
@@ -66,6 +66,7 @@
       <plugin id="org.eclipse.osgi.services"/>
       <plugin id="org.eclipse.text"/>
       <plugin id="org.eclipse.xtext.xbase.lib"/>
+      <plugin id="org.eclipse.team.core"/>
    </plugins>
 
    <configurations>

The first time it would always fail. Afterwards, I assume it'd succeed because the team.core bundle from vscode-pde got properly detected by the framework, hence the singleton warnings. I would then reproduce the behaviour by calling rm -rf server/config_linux/org.eclipse.* on the Equinox state files.

@snjeza
Copy link
Contributor Author

snjeza commented Nov 9, 2023

@rgrunber I have updated the PR.

Copy link
Contributor

@rgrunber rgrunber left a comment

Choose a reason for hiding this comment

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

I would just update the commit message and error message. After that, feel free to merge. Can you make it :

Avoid refreshing bundles whose dependency closure includes JDT-LS.

Just a note, we might be able to keep track of this in the future since the logs mention the failure. It isn't harmful though as the exception gets caught and doesn't break initialization.

@snjeza snjeza changed the title Don't refresh org.eclipse.jdt.ls.core Avoid refreshing bundles whose dependency closure includes JDT-LS. Nov 10, 2023
@rgrunber rgrunber merged commit 86f0134 into eclipse-jdtls:master Nov 10, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

is it is expected to have 19 GB of 1.1 mb log files in worksapceStorage
3 participants