opentelemetry-java-instrumentation
opentelemetry-java-instrumentation copied to clipboard
Making IDE's indexing faster
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.
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?
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.
- 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:
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.