-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
CHE-6114: Fix problems with project explorer performance in big projects #6175
Conversation
Build # 3579 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/3579/ to view the results. |
👍 |
@riuvshin can we have it in bugfix release? |
@tolusha Reworked |
Build # 3608 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/3608/ to view the results. |
Checking QA results |
@vparfonov will this be merged in this sprint? |
@bmicklea Selenium tests passed, I am going to merge it today |
Build # 3642 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/3642/ to view the results. |
Build # 3643 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/3643/ to view the results. |
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/3645/ |
Thanks for all the work on this @vinokurig |
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/3647/ |
What does this PR do?
format
parameter from get status method in Git Service, because it is redundant and do not affects to anything, since we use Jgit as a Git engine.The main goal of calling git status with filter is to increase performance. Git status request without filter might be slow if project contains a lot of files.
What issues does this PR fix or reference?
fixes #6114
Changelog
Add ability to get Git status with filter
Release Notes
Fixed problems with project explorer performance in big project.
Docs PR
N/A