Skip to content

Update fsmonitor perf suite to support integration comparisons

Nipunn Koorapati via GitGitGadget git 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest10 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/10t/perf/fsmonitor: separate one time repo initializationt/perf/p7519-fsmonitor.sh+11 -80 replies
2/10t/perf/fsmonitor: move watchman setup to one-time-repo-setupt/perf/p7519-fsmonitor.sh+9 -70 replies
3/10t/perf/fsmonitor: improve error message if typoing hook namet/perf/p7519-fsmonitor.sh+3 -12 replies
4/10t/perf/fsmonitor: factor description out for readabilityt/perf/p7519-fsmonitor.sh+12 -100 replies
5/10t/perf/fsmonitor: shorten DESC to basenamet/perf/p7519-fsmonitor.sh+5 -10 replies
6/10t/perf/fsmonitor: silence initial git committ/perf/p7519-fsmonitor.sh+1 -10 replies
7/10t/perf/fsmonitor: factor setup for fsmonitor into functiont/perf/p7519-fsmonitor.sh+6 -20 replies
8/10t/perf/fsmonitor: initialize test with git resett/perf/p7519-fsmonitor.sh+6 -20 replies
9/10t/perf/fsmonitor: perf comparison of multiple fsmonitor integrationst/perf/p7519-fsmonitor.sh+14 -80 replies
10/10t/perf/fsmonitor: add benchmark for dirty statust/perf/p7519-fsmonitor.sh+5 -00 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 00/10] Update fsmonitor perf suite to support integration comparisons

This patch series builds upon nk/diff-files-vs-fsmonitor

This builds up to a comparison between our perl script and
https://github.com/jgavris/rs-git-fsmonitor. Stats on the comparison are in the final commit message. I've found that rs-git-fsmonitor saves 20-30ms off of every git command compared to the perl script.

It may provide some motivation for supplying a faster implementation of fsmonitor-watchman.

Nipunn Koorapati (10):
t/perf/fsmonitor: separate one time repo initialization t/perf/fsmonitor: move watchman setup to one-time-repo-setup t/perf/fsmonitor: improve error message if typoing hook name t/perf/fsmonitor: factor description out for readability t/perf/fsmonitor: shorten DESC to basename t/perf/fsmonitor: silence initial git commit t/perf/fsmonitor: factor setup for fsmonitor into function t/perf/fsmonitor: initialize test with git reset t/perf/fsmonitor: perf comparison of multiple fsmonitor integrations t/perf/fsmonitor: add benchmark for dirty status

 t/perf/p7519-fsmonitor.sh | 94 ++++++++++++++++++++++++++-------------
 1 file changed, 63 insertions(+), 31 deletions(-)

base-commit: 2bfa953e5daf3253cc5fae2de2c68fbd206dfe12
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-772%2Fnipunn1313%2Fnk%2Ffsmonitor-perf-suite-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-772/nipunn1313/nk/fsmonitor-perf-suite-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/772

Signature · 1 line
gitgitgadget