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.

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

Performance regression #466

Closed
enzadesai opened this issue Jan 2, 2020 · 4 comments
Closed

Performance regression #466

enzadesai opened this issue Jan 2, 2020 · 4 comments

Comments

@enzadesai
Copy link

I just updated from 5.34.0 to 5.35.1 on IntelliJ IDEA 2019.3.1 Build 193.5662.53. There is a massive regression in performance, so much so that typing would get stuck for seconds (2-3s at times).

I am unsure what details (plugin logs, etc.) that would be useful here. I am also not certain if there are ways to instrument the plugin on my machine.

@jshiell
Copy link
Owner

jshiell commented Jan 2, 2020

Thanks for the report. Is 5.35.0 okay? There was a change in the thread handling for 5.35.1 to fix a deadlock with newer versions of IDEA, but this would result in coarser thread locking, and I was only able to test it on an i7 so it could well be the cause.

@jshiell
Copy link
Owner

jshiell commented Jan 2, 2020

Could you please try this pre-release of 5.35.2? This restores the background thread for the Checkstyle execution, but shuffles the file creation to try and avoid the deadlock in #462. Thank you!

(link removed as now released)

jshiell added a commit that referenced this issue Jan 2, 2020
@enzadesai
Copy link
Author

That was so fast! Been testing this for a bit and looks great! 👍🚢

@jshiell
Copy link
Owner

jshiell commented Jan 2, 2020

Luckily I'm still on holiday - thanks very much for the report, and sorry for the hassle!

@jshiell jshiell closed this as completed Jan 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants