* [PATCH] document git-repack interaction of pack.threads and pack.windowMemory
@ 2016-08-09 11:53 Michael Stahl
2016-08-09 18:27 ` Junio C Hamano
0 siblings, 1 reply; 3+ messages in thread
From: Michael Stahl @ 2016-08-09 11:53 UTC (permalink / raw)
To: git
Signed-off-by: Michael Stahl <mstahl@redhat.com>
---
Documentation/git-pack-objects.txt | 2 +-
Documentation/git-repack.txt | 6 +++++-
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/Documentation/git-pack-objects.txt b/Documentation/git-pack-objects.txt
index 19cdcd0..0b655a5 100644
--- a/Documentation/git-pack-objects.txt
+++ b/Documentation/git-pack-objects.txt
@@ -105,7 +105,7 @@ base-name::
advantage of the large window for the smaller objects. The
size can be suffixed with "k", "m", or "g".
`--window-memory=0` makes memory usage unlimited, which is the
- default.
+ default, unless the config variable `pack.windowMemory` is set.
--max-pack-size=<n>::
Maximum size of each output pack file. The size can be suffixed with
diff --git a/Documentation/git-repack.txt b/Documentation/git-repack.txt
index c597523..300455b 100644
--- a/Documentation/git-repack.txt
+++ b/Documentation/git-repack.txt
@@ -101,7 +101,11 @@ other objects in that pack they already have locally.
advantage of the large window for the smaller objects. The
size can be suffixed with "k", "m", or "g".
`--window-memory=0` makes memory usage unlimited, which is the
- default.
+ default, unless the config variable `pack.windowMemory` is set.
+ Note that the actual memory usage will be multiplied
+ by the number of threads used by linkgit:git-pack-objects[1],
+ which is lacking a corresponding git-repack flag but can be
+ set via the config variable `pack.threads`.
--max-pack-size=<n>::
Maximum size of each output pack file. The size can be suffixed with
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] document git-repack interaction of pack.threads and pack.windowMemory
2016-08-09 11:53 [PATCH] document git-repack interaction of pack.threads and pack.windowMemory Michael Stahl
@ 2016-08-09 18:27 ` Junio C Hamano
2016-08-10 10:39 ` Michael Stahl
0 siblings, 1 reply; 3+ messages in thread
From: Junio C Hamano @ 2016-08-09 18:27 UTC (permalink / raw)
To: Michael Stahl; +Cc: git
Michael Stahl <mstahl@redhat.com> writes:
> Signed-off-by: Michael Stahl <mstahl@redhat.com>
> ---
> Documentation/git-pack-objects.txt | 2 +-
> Documentation/git-repack.txt | 6 +++++-
> 2 files changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/git-pack-objects.txt b/Documentation/git-pack-objects.txt
> index 19cdcd0..0b655a5 100644
> --- a/Documentation/git-pack-objects.txt
> +++ b/Documentation/git-pack-objects.txt
> @@ -105,7 +105,7 @@ base-name::
> advantage of the large window for the smaller objects. The
> size can be suffixed with "k", "m", or "g".
> `--window-memory=0` makes memory usage unlimited, which is the
> - default.
> + default, unless the config variable `pack.windowMemory` is set.
That does not go far enough as it does not say what happens when the
variable is set (i.e. "the variable serves as the default in that
case").
Perhaps "...makes memory usage unlimited. The default is taken from
the `pack.windowMemory` configuration variable." would make it better,
as it is described that its default is "unlimited".
> diff --git a/Documentation/git-repack.txt b/Documentation/git-repack.txt
> index c597523..300455b 100644
> --- a/Documentation/git-repack.txt
> +++ b/Documentation/git-repack.txt
> @@ -101,7 +101,11 @@ other objects in that pack they already have locally.
> advantage of the large window for the smaller objects. The
> size can be suffixed with "k", "m", or "g".
> `--window-memory=0` makes memory usage unlimited, which is the
> - default.
> + default, unless the config variable `pack.windowMemory` is set.
> + Note that the actual memory usage will be multiplied
> + by the number of threads used by linkgit:git-pack-objects[1],
> + which is lacking a corresponding git-repack flag but can be
> + set via the config variable `pack.threads`.
I think the same comment applies to the first line. The next two
lines are definite improvement, but I do not think the last two
lines are necessary.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] document git-repack interaction of pack.threads and pack.windowMemory
2016-08-09 18:27 ` Junio C Hamano
@ 2016-08-10 10:39 ` Michael Stahl
0 siblings, 0 replies; 3+ messages in thread
From: Michael Stahl @ 2016-08-10 10:39 UTC (permalink / raw)
To: Junio C Hamano; +Cc: git
Signed-off-by: Michael Stahl <mstahl@redhat.com>
---
Documentation/git-pack-objects.txt | 4 ++--
Documentation/git-repack.txt | 6 ++++--
2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/Documentation/git-pack-objects.txt b/Documentation/git-pack-objects.txt
index 19cdcd0..8973510 100644
--- a/Documentation/git-pack-objects.txt
+++ b/Documentation/git-pack-objects.txt
@@ -104,8 +104,8 @@ base-name::
out of memory with a large window, but still be able to take
advantage of the large window for the smaller objects. The
size can be suffixed with "k", "m", or "g".
- `--window-memory=0` makes memory usage unlimited, which is the
- default.
+ `--window-memory=0` makes memory usage unlimited. The default
+ is taken from the `pack.windowMemory` configuration variable.
--max-pack-size=<n>::
Maximum size of each output pack file. The size can be suffixed with
diff --git a/Documentation/git-repack.txt b/Documentation/git-repack.txt
index c597523..26afe6e 100644
--- a/Documentation/git-repack.txt
+++ b/Documentation/git-repack.txt
@@ -100,8 +100,10 @@ other objects in that pack they already have locally.
out of memory with a large window, but still be able to take
advantage of the large window for the smaller objects. The
size can be suffixed with "k", "m", or "g".
- `--window-memory=0` makes memory usage unlimited, which is the
- default.
+ `--window-memory=0` makes memory usage unlimited. The default
+ is taken from the `pack.windowMemory` configuration variable.
+ Note that the actual memory usage will be the limit multiplied
+ by the number of threads used by linkgit:git-pack-objects[1].
--max-pack-size=<n>::
Maximum size of each output pack file. The size can be suffixed with
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-08-10 18:58 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-09 11:53 [PATCH] document git-repack interaction of pack.threads and pack.windowMemory Michael Stahl
2016-08-09 18:27 ` Junio C Hamano
2016-08-10 10:39 ` Michael Stahl
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.