Skip to content

Reduce the unnecessary refresh when editing a Java file #283

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.

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

Closed
testforstephen opened this issue Jul 23, 2020 · 0 comments · Fixed by #293
Closed

Reduce the unnecessary refresh when editing a Java file #283

testforstephen opened this issue Jul 23, 2020 · 0 comments · Fixed by #293
Labels
eng engineering
Milestone

Comments

@testforstephen
Copy link
Contributor

When i'm editing a Java file, every time i typed some characters, there are a bunch of workspace commands triggered by the Project Manager for Java extension. Too many commands probably slow down the responsiveness of the language server. Need tune the refresh workflow to avoid unnecessary refresh.

Below are the command sequence triggered at each editing:
java.resolvePath - workspace/executeCommand
java.project.list - workspace/executeCommand
java.getPackageData - workspace/executeCommand
java.getPackageData - workspace/executeCommand
java.getPackageData - workspace/executeCommand
java.getPackageData - workspace/executeCommand
java.getPackageData - workspace/executeCommand
java.getPackageData - workspace/executeCommand

java.getPackageData - workspace/executeCommand
java.getPackageData - workspace/executeCommand
java.getPackageData - workspace/executeCommand
java.getPackageData - workspace/executeCommand
java.getPackageData - workspace/executeCommand
java.getPackageData - workspace/executeCommand

@jdneo jdneo added the eng engineering label Aug 6, 2020
@jdneo jdneo added this to the 0.12.0 milestone Aug 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eng engineering
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants