Skip to content

Send 'microprofile/propertiesChanged' notification with debounce mode#541

Merged
datho7561 merged 1 commit into
eclipse-lsp4mp:masterfrom
angelozerr:debounce_propertiesChanged_notif
Jan 27, 2026
Merged

Send 'microprofile/propertiesChanged' notification with debounce mode#541
datho7561 merged 1 commit into
eclipse-lsp4mp:masterfrom
angelozerr:debounce_propertiesChanged_notif

Conversation

@angelozerr
Copy link
Copy Markdown
Contributor

@angelozerr angelozerr commented Jan 27, 2026

Send 'microprofile/propertiesChanged' notification with debounce mode

See redhat-developer/vscode-quarkus#1204

This PR provides this new behavior: when Java file is saved, 'microprofile/propertiesChanged' is sent after 2 sec. It means if you save N files in same time, only one 'microprofile/propertiesChanged' should be sent instead of N notifications.

When 'microprofile/propertiesChanged' is send from JDT to LS, the MicroProfile LS request the projectInfo request is called (to collect MP/Quarkus properties). It will avoid calling N projectInfo when N Java files are saved in the same time.

I hope it will fix the redhat-developer/vscode-quarkus#1204 issue

Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

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

Looks good. I confirmed in the messages that the debouncing is happening. Hopefullly it fixes or helps with the linked issue.

@datho7561 datho7561 merged commit 5b16598 into eclipse-lsp4mp:master Jan 27, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants