From: Gumbel, Matthew K Date: Fri, 03 Feb 2017 23:22:30 GMT Subject: [PATCH] commit: Optimize number of lstat() calls Message-ID: When making a --only commit, original behavior was to do a full cache update for the purposes of giving the pre-commit hook an up-to-date set of stat data. That would result in long runtime for git-commit in a big repo on NFS (>60s for a 54k-file repo). With this change, when doing a --only commit and no pre-commit hook is present, the cache update is skipped since it is known a priori which files are to be committed. This was discussed on the mailing list here: https://public-inbox.org/git/DA0A42D68346B1469147552440A645039A9C56D4@ORSMSX115.amr.corp.intel.com/ Signed-off-by: Matthew K. Gumbel --- builtin/commit.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/builtin/commit.c b/builtin/commit.c index 2de5f6c..1df3d71 100644 --- a/builtin/commit.c +++ b/builtin/commit.c @@ -470,7 +470,9 @@ static const char *prepare_index(int argc, const char **argv, const char *prefix     hold_locked_index(&index_lock, LOCK_DIE_ON_ERROR);     add_remove_files(&partial); -   refresh_cache(REFRESH_QUIET); +    if (find_hook("pre-commit")) { +        refresh_cache(REFRESH_QUIET); +    }     update_main_cache_tree(WRITE_TREE_SILENT);     if (write_locked_index(&the_index, &index_lock, CLOSE_LOCK))         die(_("unable to write new_index file")); @@ -482,7 +484,9 @@ static const char *prepare_index(int argc, const char **argv, const char *prefix     create_base_index(current_head);     add_remove_files(&partial); -   refresh_cache(REFRESH_QUIET); +    if (find_hook("pre-commit")) { +        refresh_cache(REFRESH_QUIET); +    }     if (write_locked_index(&the_index, &false_lock, CLOSE_LOCK))         die(_("unable to write temporary index file")); -- 2.8.4