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

Add MultiThread TreeWalker implementation #4957

Open
soon opened this issue Aug 17, 2017 · 3 comments
Open

Add MultiThread TreeWalker implementation #4957

soon opened this issue Aug 17, 2017 · 3 comments

Comments

@soon
Copy link
Contributor

soon commented Aug 17, 2017

Since walking is one of the most consuming operations, we should process checks in parallel inside the MT version of TreeWalker

@jo2
Copy link
Contributor

jo2 commented Nov 12, 2019

Can someone give a status update here? I'm running into multiple problems with synchronization in version 8.26. The GitHub project mentioned here is closed but the issue is stil open and in ToDO..

If there is no work in progress I work on some PRs fixing the problems I'll run into..

@romani
Copy link
Member

romani commented Nov 12, 2019

Checkstyle is single thread tool now, we did not come on agreement on MT mode, so postponed such discussion till we have time to look at it one more time.

@rnveach
Copy link
Member

rnveach commented Nov 12, 2019

@jo2 since there is no multi-threading right now, I recommend creating a new issue for your problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Development

No branches or pull requests

5 participants