From: Neeraj Singh <neerajsi@xxxxxxxxxxxxx>
Add basic performance tests for "git add" and "git stash" of a lot of
new objects with various fsync settings. This shows the benefit of batch
mode relative to an ordinary stash command.
Signed-off-by: Neeraj Singh <neerajsi@xxxxxxxxxxxxx>
---
t/perf/p3700-add.sh | 59 ++++++++++++++++++++++++++++++++++++++++
t/perf/p3900-stash.sh | 62 +++++++++++++++++++++++++++++++++++++++++++
t/perf/perf-lib.sh | 4 +--
3 files changed, 123 insertions(+), 2 deletions(-)
create mode 100755 t/perf/p3700-add.sh
create mode 100755 t/perf/p3900-stash.sh
diff --git a/t/perf/p3700-add.sh b/t/perf/p3700-add.sh
new file mode 100755
index 00000000000..2ea78c9449d
--- /dev/null
+++ b/t/perf/p3700-add.sh
@@ -0,0 +1,59 @@
+#!/bin/sh
+#
+# This test measures the performance of adding new files to the object database
+# and index. The test was originally added to measure the effect of the
+# core.fsyncMethod=batch mode, which is why we are testing different values
+# of that setting explicitly and creating a lot of unique objects.
+
+test_description="Tests performance of add"
+
+# Fsync is normally turned off for the test suite.
+GIT_TEST_FSYNC=1
+export GIT_TEST_FSYNC
+
+. ./perf-lib.sh
+
+. $TEST_DIRECTORY/lib-unique-files.sh
+
+test_perf_default_repo
+test_checkout_worktree
+
+dir_count=10
+files_per_dir=50
+total_files=$((dir_count * files_per_dir))
+
+# We need to create the files each time we run the perf test, but
+# we do not want to measure the cost of creating the files, so run
+# the test once.
+if test "${GIT_PERF_REPEAT_COUNT-1}" -ne 1
+then
+ echo "warning: Setting GIT_PERF_REPEAT_COUNT=1" >&2
+ GIT_PERF_REPEAT_COUNT=1
+fi
+
+for m in false true batch
+do
+ test_expect_success "create the files for object_fsyncing=$m" '
+ git reset --hard &&
+ # create files across directories
+ test_create_unique_files $dir_count $files_per_dir files
+ '
+
+ case $m in
+ false)
+ FSYNC_CONFIG='-c core.fsync=-loose-object -c core.fsyncmethod=fsync'
+ ;;
+ true)
+ FSYNC_CONFIG='-c core.fsync=loose-object -c core.fsyncmethod=fsync'
+ ;;
+ batch)
+ FSYNC_CONFIG='-c core.fsync=loose-object -c core.fsyncmethod=batch'
+ ;;
+ esac
+
+ test_perf "add $total_files files (object_fsyncing=$m)" "
+ git $FSYNC_CONFIG add files
+ "
+done
+
+test_done
diff --git a/t/perf/p3900-stash.sh b/t/perf/p3900-stash.sh
new file mode 100755
index 00000000000..3526f06cef4
--- /dev/null
+++ b/t/perf/p3900-stash.sh
@@ -0,0 +1,62 @@
+#!/bin/sh
+#
+# This test measures the performance of adding new files to the object database
+# and index. The test was originally added to measure the effect of the
+# core.fsyncMethod=batch mode, which is why we are testing different values
+# of that setting explicitly and creating a lot of unique objects.
+
+test_description="Tests performance of stash"
+
+# Fsync is normally turned off for the test suite.
+GIT_TEST_FSYNC=1
+export GIT_TEST_FSYNC
+
+. ./perf-lib.sh
+
+. $TEST_DIRECTORY/lib-unique-files.sh
+
+test_perf_default_repo
+test_checkout_worktree
+
+dir_count=10
+files_per_dir=50
+total_files=$((dir_count * files_per_dir))
+
+# We need to create the files each time we run the perf test, but
+# we do not want to measure the cost of creating the files, so run
+# the test once.
+if test "${GIT_PERF_REPEAT_COUNT-1}" -ne 1
+then
+ echo "warning: Setting GIT_PERF_REPEAT_COUNT=1" >&2
+ GIT_PERF_REPEAT_COUNT=1
+fi
+
+for m in false true batch
+do
+ test_expect_success "create the files for object_fsyncing=$m" '
+ git reset --hard &&
+ # create files across directories
+ test_create_unique_files $dir_count $files_per_dir files
+ '
+
+ case $m in
+ false)
+ FSYNC_CONFIG='-c core.fsync=-loose-object -c core.fsyncmethod=fsync'
+ ;;
+ true)
+ FSYNC_CONFIG='-c core.fsync=loose-object -c core.fsyncmethod=fsync'
+ ;;
+ batch)
+ FSYNC_CONFIG='-c core.fsync=loose-object -c core.fsyncmethod=batch'
+ ;;
+ esac
+
+ # We only stash files in the 'files' subdirectory since
+ # the perf test infrastructure creates files in the
+ # current working directory that need to be preserved
+ test_perf "stash $total_files files (object_fsyncing=$m)" "
+ git $FSYNC_CONFIG stash push -u -- files
+ "
+done
+
+test_done
diff --git a/t/perf/perf-lib.sh b/t/perf/perf-lib.sh
index 932105cd12c..d270d1d962a 100644
--- a/t/perf/perf-lib.sh
+++ b/t/perf/perf-lib.sh
@@ -98,8 +98,8 @@ test_perf_create_repo_from () {
mkdir -p "$repo/.git"
(
cd "$source" &&
- { cp -Rl "$objects_dir" "$repo/.git/" 2>/dev/null ||
- cp -R "$objects_dir" "$repo/.git/"; } &&
+ { cp -Rl "$objects_dir" "$repo/.git/" ||
+ cp -R "$objects_dir" "$repo/.git/" 2>/dev/null;} &&
# common_dir must come first here, since we want source_git to
# take precedence and overwrite any overlapping files
--
gitgitgadget
