-
Notifications
You must be signed in to change notification settings - Fork 1
Description
Related issues:
- Synchronization of bazel projects (performance) #9
- Cache project discovery (performance) #13
- Bazel Eclipse doesn't recognize filegroup #15
I have started with https://github.com/eclipseguru/bazel-eclipse/tree/performance, commit 8a6f0db bazel-eclipse, the main branch and tried to import large to Eclipse using Bazel Eclipse Feature (BEF)
There are several issues:
-
The project has been created for Bazel 5, but IntelliJ aspects require Bazel>= 6
I have updated the project to work with Bazel 6.1.0. so that now IntelliJ aspects run correctly. -
The project uses
java/resourcesreferences that aren’t usual for Bazel - Bazel Eclipse doesn't recognize filegroup #15 (comment)
I have replaced them withresources. -
I have added the PR Bazel Eclipse doesn't recognize filegroup #15
When I try to import a project, that takes a long time or Eclipse freezes.
- The main issue is that Bazel Eclipse refreshes projects on
Line 89 in 9f8988a
refreshResources(resourcesToRefresh, monitor);
that causes it to repeatedly try to configure projects.
I have commented this call. The Sync Bazel Projects View action happens in the workspace job. Eclipse will automatically refresh projects when the job is finished.
@guw Is there any special reason to call refreshLocal within the workspace job?
-
The
findProjectmethod is still called multiple times for the same package/target. -
JvmConfiguratorwill dispose and recreate VM every time theSync Bazel Projects Viewaction is called.
This is a patch against the https://github.com/eclipseguru/bazel-eclipse/tree/performance branch
If the patch is included, importing https://github.com/snjeza/bazel-ls-demo-project/tree/master/large will take about 10 minutes
@guw I would suggest you to apply your #13.
After that, I will create a PR for the issues described in this issue.