* [PATCH] Document git-svn fetch --log-window-size parameter
@ 2012-10-23 10:33 Gunnlaugur Þór Briem
2012-10-25 9:52 ` Jeff King
0 siblings, 1 reply; 8+ messages in thread
From: Gunnlaugur Þór Briem @ 2012-10-23 10:33 UTC (permalink / raw)
To: git; +Cc: Gunnlaugur Þór Briem
The --log-window-size parameter to git-svn fetch is undocumented.
Minimally describe what it does and why the user might change it.
Signed-off-by: Gunnlaugur Þór Briem <gunnlaugur@gmail.com>
---
Documentation/git-svn.txt | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/Documentation/git-svn.txt b/Documentation/git-svn.txt
index cfe8d2b..64756c9 100644
--- a/Documentation/git-svn.txt
+++ b/Documentation/git-svn.txt
@@ -146,6 +146,12 @@ Skip "branches" and "tags" of first level directories;;
------------------------------------------------------------------------
--
+--log-window-size=<n>;;
+ Fetch <n> log entries per request when scanning Subversion history.
+ The default is 100. For very large Subversion repositories, larger
+ values may be needed for 'clone'/'fetch' to complete in reasonable
+ time. But overly large values may lead to request timeouts.
+
'clone'::
Runs 'init' and 'fetch'. It will automatically create a
directory based on the basename of the URL passed to it;
--
1.7.12.3
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] Document git-svn fetch --log-window-size parameter
2012-10-23 10:33 [PATCH] Document git-svn fetch --log-window-size parameter Gunnlaugur Þór Briem
@ 2012-10-25 9:52 ` Jeff King
2012-10-26 9:46 ` Eric Wong
0 siblings, 1 reply; 8+ messages in thread
From: Jeff King @ 2012-10-25 9:52 UTC (permalink / raw)
To: Gunnlaugur Þór Briem; +Cc: Eric Wong, git
On Tue, Oct 23, 2012 at 10:33:26AM +0000, Gunnlaugur Þór Briem wrote:
> The --log-window-size parameter to git-svn fetch is undocumented.
>
> Minimally describe what it does and why the user might change it.
Thanks. Your description makes sense to me, but I do not have enough
git-svn knowledge to know if it covers all intended uses of the flag.
Eric?
> +--log-window-size=<n>;;
> + Fetch <n> log entries per request when scanning Subversion history.
> + The default is 100. For very large Subversion repositories, larger
> + values may be needed for 'clone'/'fetch' to complete in reasonable
> + time. But overly large values may lead to request timeouts.
-Peff
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Document git-svn fetch --log-window-size parameter
2012-10-25 9:52 ` Jeff King
@ 2012-10-26 9:46 ` Eric Wong
2012-10-26 13:32 ` Jeff King
0 siblings, 1 reply; 8+ messages in thread
From: Eric Wong @ 2012-10-26 9:46 UTC (permalink / raw)
To: Gunnlaugur Þór Briem; +Cc: Jeff King, git
Jeff King <peff@peff.net> wrote:
> On Tue, Oct 23, 2012 at 10:33:26AM +0000, Gunnlaugur Þór Briem wrote:
>
> > The --log-window-size parameter to git-svn fetch is undocumented.
> >
> > Minimally describe what it does and why the user might change it.
>
> Thanks. Your description makes sense to me, but I do not have enough
> git-svn knowledge to know if it covers all intended uses of the flag.
> Eric?
>
> > +--log-window-size=<n>;;
> > + Fetch <n> log entries per request when scanning Subversion history.
> > + The default is 100. For very large Subversion repositories, larger
> > + values may be needed for 'clone'/'fetch' to complete in reasonable
> > + time. But overly large values may lead to request timeouts.
Overly large values also lead to excessive memory usage. I may have
only had 256M in my dev machine at the time I added this parameter:
commit 6af1db447b10c03db4c04a55000efaa9aad38caa
Author: Eric Wong <normalperson@yhbt.net>
Date: Wed Feb 14 16:04:10 2007 -0800
git-svn: allow --log-window-size to be specified, default to 100
The newer default value should should lower memory usage for
large fetches and also help with fetching from less reliable
servers. Previously the value was 1000 and memory usage
got a bit high on some repositories and fetching became
less reliable in some cases.
Signed-off-by: Eric Wong <normalperson@yhbt.net>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Document git-svn fetch --log-window-size parameter
2012-10-26 9:46 ` Eric Wong
@ 2012-10-26 13:32 ` Jeff King
2012-10-26 13:45 ` Gunnlaugur Thor Briem
2012-10-29 20:18 ` Eric Wong
0 siblings, 2 replies; 8+ messages in thread
From: Jeff King @ 2012-10-26 13:32 UTC (permalink / raw)
To: Eric Wong; +Cc: Gunnlaugur Þór Briem, git
On Fri, Oct 26, 2012 at 09:46:02AM +0000, Eric Wong wrote:
> > Thanks. Your description makes sense to me, but I do not have enough
> > git-svn knowledge to know if it covers all intended uses of the flag.
> > Eric?
> >
> > > +--log-window-size=<n>;;
> > > + Fetch <n> log entries per request when scanning Subversion history.
> > > + The default is 100. For very large Subversion repositories, larger
> > > + values may be needed for 'clone'/'fetch' to complete in reasonable
> > > + time. But overly large values may lead to request timeouts.
>
> Overly large values also lead to excessive memory usage. I may have
> only had 256M in my dev machine at the time I added this parameter:
That's probably worth mentioning. Gunnlaugur, any objection to me
amending your commit with:
diff --git a/Documentation/git-svn.txt b/Documentation/git-svn.txt
index 64756c9..8b0d3ad 100644
--- a/Documentation/git-svn.txt
+++ b/Documentation/git-svn.txt
@@ -150,7 +150,8 @@ Skip "branches" and "tags" of first level directories;;
Fetch <n> log entries per request when scanning Subversion history.
The default is 100. For very large Subversion repositories, larger
values may be needed for 'clone'/'fetch' to complete in reasonable
- time. But overly large values may lead to request timeouts.
+ time. But overly large values may lead to higher memory usage and
+ request timeouts.
'clone'::
Runs 'init' and 'fetch'. It will automatically create a
?
-Peff
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] Document git-svn fetch --log-window-size parameter
2012-10-26 13:32 ` Jeff King
@ 2012-10-26 13:45 ` Gunnlaugur Thor Briem
2012-10-29 20:18 ` Eric Wong
1 sibling, 0 replies; 8+ messages in thread
From: Gunnlaugur Thor Briem @ 2012-10-26 13:45 UTC (permalink / raw)
To: Jeff King; +Cc: Eric Wong, git
On Fri, Oct 26, 2012 at 1:32 PM, Jeff King <peff@peff.net> wrote:
> That's probably worth mentioning. Gunnlaugur, any objection to me
> amending your commit with:
>
> diff --git a/Documentation/git-svn.txt b/Documentation/git-svn.txt
> index 64756c9..8b0d3ad 100644
> --- a/Documentation/git-svn.txt
> +++ b/Documentation/git-svn.txt
> @@ -150,7 +150,8 @@ Skip "branches" and "tags" of first level directories;;
> Fetch <n> log entries per request when scanning Subversion history.
> The default is 100. For very large Subversion repositories, larger
> values may be needed for 'clone'/'fetch' to complete in reasonable
> - time. But overly large values may lead to request timeouts.
> + time. But overly large values may lead to higher memory usage and
> + request timeouts.
>
> 'clone'::
> Runs 'init' and 'fetch'. It will automatically create a
>
> ?
No objection, that sounds fine to me.
-Gulli
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Document git-svn fetch --log-window-size parameter
2012-10-26 13:32 ` Jeff King
2012-10-26 13:45 ` Gunnlaugur Thor Briem
@ 2012-10-29 20:18 ` Eric Wong
2012-10-29 22:02 ` Jeff King
1 sibling, 1 reply; 8+ messages in thread
From: Eric Wong @ 2012-10-29 20:18 UTC (permalink / raw)
To: Jeff King; +Cc: Gunnlaugur Þór Briem, git
Jeff King <peff@peff.net> wrote:
> On Fri, Oct 26, 2012 at 09:46:02AM +0000, Eric Wong wrote:
> > Overly large values also lead to excessive memory usage. I may have
> > only had 256M in my dev machine at the time I added this parameter:
>
> That's probably worth mentioning. Gunnlaugur, any objection to me
> amending your commit with:
Thanks both, I've amended and S-o-b on my end. Shall I add:
Signed-off-by: Jeff King <peff@peff.net>
and push?
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Document git-svn fetch --log-window-size parameter
2012-10-29 20:18 ` Eric Wong
@ 2012-10-29 22:02 ` Jeff King
2012-10-29 22:36 ` Eric Wong
0 siblings, 1 reply; 8+ messages in thread
From: Jeff King @ 2012-10-29 22:02 UTC (permalink / raw)
To: Eric Wong; +Cc: Gunnlaugur Þór Briem, git
On Mon, Oct 29, 2012 at 08:18:47PM +0000, Eric Wong wrote:
> Jeff King <peff@peff.net> wrote:
> > On Fri, Oct 26, 2012 at 09:46:02AM +0000, Eric Wong wrote:
> > > Overly large values also lead to excessive memory usage. I may have
> > > only had 256M in my dev machine at the time I added this parameter:
> >
> > That's probably worth mentioning. Gunnlaugur, any objection to me
> > amending your commit with:
>
> Thanks both, I've amended and S-o-b on my end. Shall I add:
> Signed-off-by: Jeff King <peff@peff.net>
Since it was such a small topic, I just went ahead and queued bc22b27 in
my repo. If that's not OK, I can revert it from 'next' and wait to get
it by pulling from you.
-Peff
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Document git-svn fetch --log-window-size parameter
2012-10-29 22:02 ` Jeff King
@ 2012-10-29 22:36 ` Eric Wong
0 siblings, 0 replies; 8+ messages in thread
From: Eric Wong @ 2012-10-29 22:36 UTC (permalink / raw)
To: Jeff King; +Cc: Gunnlaugur Þór Briem, git
Jeff King <peff@peff.net> wrote:
> Since it was such a small topic, I just went ahead and queued bc22b27 in
> my repo. If that's not OK, I can revert it from 'next' and wait to get
> it by pulling from you.
That's fine, thanks!
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-10-29 22:36 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-10-23 10:33 [PATCH] Document git-svn fetch --log-window-size parameter Gunnlaugur Þór Briem
2012-10-25 9:52 ` Jeff King
2012-10-26 9:46 ` Eric Wong
2012-10-26 13:32 ` Jeff King
2012-10-26 13:45 ` Gunnlaugur Thor Briem
2012-10-29 20:18 ` Eric Wong
2012-10-29 22:02 ` Jeff King
2012-10-29 22:36 ` Eric Wong
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).