Skip to content

LSP: Speed up publish diagnostics on project scan.#7737

Merged
dbalek merged 1 commit intoapache:masterfrom
dbalek:dbalek/lsp-publish-diags2
Sep 27, 2024
Merged

LSP: Speed up publish diagnostics on project scan.#7737
dbalek merged 1 commit intoapache:masterfrom
dbalek:dbalek/lsp-publish-diags2

Conversation

@dbalek
Copy link
Copy Markdown
Contributor

@dbalek dbalek commented Sep 6, 2024

An attempt to speed up publishing diagnostics collected on project scan by directly publishing data from caches.

@dbalek dbalek added CSL [ci] enable web job Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) LSP [ci] enable Language Server Protocol tests VSCode Extension Micronaut [ci] enable enterprise job labels Sep 6, 2024
@dbalek dbalek added this to the NB24 milestone Sep 6, 2024
@dbalek dbalek requested a review from lahodaj September 6, 2024 17:52
@dbalek dbalek self-assigned this Sep 6, 2024
@dbalek dbalek force-pushed the dbalek/lsp-publish-diags2 branch from b10c8b6 to 4ec3882 Compare September 9, 2024 10:45
Comment thread ide/csl.api/src/org/netbeans/modules/csl/core/TLIndexerFactory.java
Comment thread ide/parsing.indexing/apichanges.xml
@dbalek dbalek force-pushed the dbalek/lsp-publish-diags2 branch from 4ec3882 to 5c7fa91 Compare September 26, 2024 09:26
@dbalek dbalek force-pushed the dbalek/lsp-publish-diags2 branch from 5c7fa91 to ec8384f Compare September 26, 2024 16:09
Copy link
Copy Markdown
Contributor

@lahodaj lahodaj left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me.

@dbalek dbalek requested a review from MartinBalin September 27, 2024 09:40
@dbalek dbalek merged commit 4a7bacc into apache:master Sep 27, 2024
@dbalek dbalek deleted the dbalek/lsp-publish-diags2 branch September 27, 2024 15:41
@mbien
Copy link
Copy Markdown
Member

mbien commented Nov 22, 2024

this seems to be causing a regression #7981

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CSL [ci] enable web job Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) LSP [ci] enable Language Server Protocol tests Micronaut [ci] enable enterprise job VSCode Extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants