opentelemetry-java-instrumentation icon indicating copy to clipboard operation
opentelemetry-java-instrumentation copied to clipboard

Making IDE's indexing faster

Open LikeTheSalad opened this issue 1 year ago • 3 comments

This is an approach to reduce the indexing time, and sometimes IDE's freezing, caused by the large amount of subprojects involved in this repo.

The idea is always to include/enable the "core" projects (the ones that are needed to make all others to work, which I'm not sure if I spotted them all), and only include the other ones when the build is running within a GitHub Action.

This means users would have to include the projects they'd like to enable locally here.

LikeTheSalad avatar Feb 02 '24 10:02 LikeTheSalad

This idea sounds similar to https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/docs/contributing/intellij-setup-and-troubleshooting.md#loadunload-modules - how does it compare?

zeitlinger avatar Feb 02 '24 11:02 zeitlinger

This idea sounds similar to https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/docs/contributing/intellij-setup-and-troubleshooting.md#loadunload-modules - how does it compare?

Oh! I didn't know something like that was already implemented, I remember some discussions around finding a solution for it in the SIG but I thought it hadn't been applied yet because I got all the projects loaded today after I fetched the latest changes 😅 - I think a solution like this one should be the default behavior tbh, I don't think there's going to be someone (or enough people) who needs to have all these projects running locally so that it should be the default behavior to have then all enabled.

how does it compare?

I just saw the link you shared and I think these are the differences:

  • The "module unload" option seem to be IntelliJ-specific? and it has the Kotlin compiler dependency. Whereas this one relies on gradle only.
  • This one is "enabled" by default.

LikeTheSalad avatar Feb 02 '24 12:02 LikeTheSalad

  • The "module unload" option seem to be IntelliJ-specific?

No, only relies on Gradle

and it has the Kotlin compiler dependency.

Yes, to run the script.

  • This one is "enabled" by default.

I really like that :smile:

zeitlinger avatar Feb 02 '24 13:02 zeitlinger

This PR has been labeled as stale due to lack of activity. It will be automatically closed if there is no further activity over the next 14 days.

github-actions[bot] avatar Sep 30 '25 23:09 github-actions[bot]