-
Notifications
You must be signed in to change notification settings - Fork 85
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
Running git-radar --bash is slow #115
Comments
Not having looked into the depths of git-radar but it is definitely fixable. I made my own "Poor man's git-radar" and had a similar issue. The trick was to bail out of any further processing if you can avoid it. eg,
As to debugging which parts of |
Definitely can be optimised. I've fell out of maintaining git-radar due to it satisfying my need but it's been nagging me that it is slow. As @neozenith says there are optimisations that can be made but ultimately Bash is slow at processing strings. I'm tempted to reimplement fairly soon using a faster language that can access the libgit2 instead, thus optimising usage of git as well. Go would probably be a good choice. |
Maintenance Also the Another option is keep it all here. I'd be happy to step up as a maintainer, triage issues, run code reviews etc. Not sure if you have elevated anyone to a maintainer status already or not. It's a really well developed project (just look at that README) and I personally think it'd be a shame to let it go quiet. I get quite a few comments from colleagues about my poor man's version that they'd like to have it too. https://github.com/neozenith/vim-dotfiles#git-bash-prompt Possible Test Cases I know a good repository that is notoriously large to test this against. YouCompleteMe autocomplete engine. There was another one that a company I was working for had python scripts generating and updating C++ files that were version controlled that would be good tests. Even having a sample repo commit |
In a repository with a .git folder that is ~30MB big, it's takes around ~6 seconds to run
git-radar --bash
which is too slow, especially when constantly opening new terminal sessions. Is there any way to speed up the command or is this a limitation that can't can't be optimized? Thanks.The text was updated successfully, but these errors were encountered: