-
Notifications
You must be signed in to change notification settings - Fork 232
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 issues on man completions #588
Comments
Out of curiosity, I'd be interested if you see a speed improvement if you compile nushell differently using this command.
or if you install from crates.io
I've noticed some fairly significant performance increases with |
Using |
What do you want from a time command, the name itself is a bit ambiguous. We have |
[monkey@monkeydragon ~]$ time du -hs ~/
956G /home/monkey/
real 0m56.042s
user 0m1.849s
sys 0m25.028s It's useful for measuring the time it takes for a script/process to complete but in particular allows you to examine how much time the program spends doing things like syscalls Vs running the program itself Vs wall (real) time. In the above example, notice that the
|
So, for results for the difference between the default allocator and the minalloc:
Default:
Minalloc:
There is definitely some extra overhead from the completions itself, but I'm not sure how to time the character press to completions becoming available, which seems to be an extra 4ish seconds on top of this. |
Basically the title. Completions can take up to a second or more, a quick count reveals I have
57124
manpage entries on my system. The initial completion taking a while may be reasonable, however, I believe what makes it not reasonable is attempting to do something along the lines ofman git-<tab>
(examine results)c
takes a few seconds to appear again, then any subsequent additional characters triggers a complete regeneration of search list. This makes typing the rest of the manpage name unreasonably slow.I have a handful of suggestions to make this better, namely caching at minimum entries in
man -w
so there is not nearly as much filesystem traversal, but I wanted to get opinions.The text was updated successfully, but these errors were encountered: