Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Lazy resolve postfix completion #2616

Closed
jdneo opened this issue Apr 24, 2023 · 0 comments · Fixed by #2618
Closed

Lazy resolve postfix completion #2616

jdneo opened this issue Apr 24, 2023 · 0 comments · Fixed by #2618
Assignees
@jdneo
Labels
content-assist performance
Milestone
End April 2023

Comments

@jdneo
Copy link
Contributor

jdneo commented Apr 24, 2023

With #2600 we can consider put more completion items to be lazy resolved.

Postfix could be a good candidate. With the code below:

public static void main(String[] args) throws Exception {
    args.|
}

Triggering completion at |, get the following profile result
image

evaluateGenericTemplate() occupies considerable CPU time during the completion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jdneo jdneo

Labels
content-assist performance
Projects
None yet
Milestone
End April 2023
Development

Successfully merging a pull request may close this issue.

Support lazy-resolving for postfix completion jdneo/eclipse.jdt.ls
1 participant
@jdneo