Skip to content

Add feature flag to Java generator to disable Eclipse Language Server usage#7444

Merged
alzimmermsft merged 4 commits into
microsoft:mainfrom
alzimmermsft:Java_RemoveEclipseLanguageServer
May 27, 2025
Merged

Add feature flag to Java generator to disable Eclipse Language Server usage#7444
alzimmermsft merged 4 commits into
microsoft:mainfrom
alzimmermsft:Java_RemoveEclipseLanguageServer

Conversation

@alzimmermsft
Copy link
Copy Markdown
Contributor

This PR is adding a feature flag use-eclipse-language-server that will control whether the Eclipse language server is used when running cod customizations. This flag will be temporary and the default value will be true. The long-term goal is for all projects using code customizations to set this value to false, and once that is done the flag will be removed as well as any Eclipse language server related code.

The rationale behind this change is that while Eclipse language server is extremely powerful it allows for too many customizations to be used. Code customizations were originally added for niche issues found in the generators and was meant to have limited usage, which is what Eclipse language server circumvents. Additionally, it requires a large download to run and is much slower than using in-memory JavaParser capabilities.

@alzimmermsft alzimmermsft self-assigned this May 22, 2025
@microsoft-github-policy-service microsoft-github-policy-service Bot added the emitter:client:java Issue for the Java client emitter: @typespec/http-client-java label May 22, 2025
@github-actions
Copy link
Copy Markdown
Contributor

No changes needing a change description found.

@azure-sdk
Copy link
Copy Markdown
Collaborator

azure-sdk commented May 22, 2025

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@alzimmermsft alzimmermsft marked this pull request as ready for review May 27, 2025 15:23
Copy link
Copy Markdown
Member

@anuchandy anuchandy left a comment

Choose a reason for hiding this comment

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

Thanks Alan, looks good to me, but let's wait for Weidong and others to have a look as well.

@alzimmermsft alzimmermsft enabled auto-merge May 27, 2025 17:52
@alzimmermsft alzimmermsft added this pull request to the merge queue May 27, 2025
Merged via the queue into microsoft:main with commit 666644d May 27, 2025
25 checks passed
@alzimmermsft alzimmermsft deleted the Java_RemoveEclipseLanguageServer branch May 27, 2025 18:26
weidongxu-microsoft added a commit that referenced this pull request May 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:client:java Issue for the Java client emitter: @typespec/http-client-java

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants