{"thread":{"id":"7415","subject":"git svn fetch spewing warnings","startedAt":"2007-03-26T17:02:50Z","lastAt":"2007-04-02T18:46:53Z","messageCount":6,"participants":["Seth Falcon","Eric Wong"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"38038","messageId":"m28xdkvsxx.fsf@ziti.fhcrc.org","threadId":"7415","inReplyTo":null,"subject":"git svn fetch spewing warnings","fromName":"Seth Falcon","fromEmail":"sethfalcon@gmail.com","sentAt":"2007-03-26T17:02:50Z","receivedAt":"2007-03-26T17:02:50Z","isPatch":false,"sender":{"key":"sethfalcon@gmail.com","avatar":"https://gravatar.com/avatar/fd62fe73d3013b12fce71d5269ec52eaca3c4cee479efc6eea3603d6f1d8bf8d?d=mp&s=160"},"body":"Hi,\n\nI'm using git 1.5.1.rc1.13.g0872 and I just tried the following to\ncreate a new git-svn tracked repository.\n\nI did:\n\n\n git svn init $URL\n git svn fetch \n # I also tried git svn fetch --no-follow-parent\n\nThe command eventually completed and left me with a working\nrepository, but in the process I had screenfuls of warning messages\nlike this:\n\nW: Ignoring error from SVN, path probably does not exist: (175002): RA layer request failed: REPORT request failed on '/bioconductor/!svn/bc/12000': REPORT of '/bioconductor/!svn/bc/12000': Could not read chunk size: Secure connection truncated (https://hedgehog.fhcrc.org)\nW: Ignoring error from SVN, path probably does not exist: (175002): RA layer request failed: REPORT request failed on '/bioconductor/!svn/bc/12100': REPORT of '/bioconductor/!svn/bc/12100': Could not read chunk size: Secure connection truncated (https://hedgehog.fhcrc.org)\nW: Ignoring error from SVN, path probably does not exist: (175002): RA layer request failed: REPORT request failed on '/bioconductor/!svn/bc/12200': REPORT of '/bioconductor/!svn/bc/12200': Could not read chunk size: Secure connection truncated (https://hedgehog.fhcrc.org)\n\nI haven't seen them before, but...\n\n 1. I haven't initialized a new repository in this way for awhile and\n git-svn has been updated\n\n 2. The subversion server I'm talking to was recently upgraded to\n 1.4.2\n\nThanks,\n\n+ seth\n-- \nSeth Falcon | Computational Biology | Fred Hutchinson Cancer Research Center\nhttp://bioconductor.org\n"},{"id":"38175","messageId":"20070327190004.GD16591@localdomain","threadId":"7415","inReplyTo":"m28xdkvsxx.fsf@ziti.fhcrc.org","subject":"Re: git svn fetch spewing warnings","fromName":"Eric Wong","fromEmail":"normalperson@yhbt.net","sentAt":"2007-03-27T19:00:04Z","receivedAt":"2007-03-27T19:00:04Z","isPatch":false,"sender":{"key":"e@80x24.org","avatar":null},"body":"Seth Falcon wrote:\n> Hi,\n> \n> I'm using git 1.5.1.rc1.13.g0872 and I just tried the following to\n> create a new git-svn tracked repository.\n> \n> I did:\n> \n> \n> git svn init $URL\n> git svn fetch \n> # I also tried git svn fetch --no-follow-parent\n> \n> The command eventually completed and left me with a working\n> repository, but in the process I had screenfuls of warning messages\n> like this:\n\nWe now try harder to find previous versions of the path we're tracking.\nI've also changed the window from 1000 to 100 revisions, so the\nwarning message spewing is 10 times more frequent. Don't worry, though\nit's harmless :)\n\n-- \nEric Wong\n"},{"id":"38176","messageId":"m23b3qts11.fsf@ziti.fhcrc.org","threadId":"7415","inReplyTo":"20070327190004.GD16591@localdomain","subject":"Re: git svn fetch spewing warnings","fromName":"Seth Falcon","fromEmail":"sethfalcon@gmail.com","sentAt":"2007-03-27T19:17:46Z","receivedAt":"2007-03-27T19:17:46Z","isPatch":false,"sender":{"key":"sethfalcon@gmail.com","avatar":"https://gravatar.com/avatar/fd62fe73d3013b12fce71d5269ec52eaca3c4cee479efc6eea3603d6f1d8bf8d?d=mp&s=160"},"body":"Eric Wong writes:\n> We now try harder to find previous versions of the path we're tracking.\n> I've also changed the window from 1000 to 100 revisions, so the\n> warning message spewing is 10 times more frequent. Don't worry, though\n> it's harmless :)\n\nNot a big deal. But perhaps those warnings could go to a log file of\nsome kind and then a summary at the end could say:\n\n There were 500 warning messages, check git-svn-fetch-TIMESTAMP.log\n for details.\n\nAnd to push a bit further, why do I care about these messages. What\naction might I take based on them? I almost didn't discover that the\nwarnings were harmless because I C-c'd out a few times thinking that\nsomething was really wrong before deciding to just let it run.\n\n+ seth\n\n-- \nSeth Falcon\n"},{"id":"38396","messageId":"20070331005448.GA6948@muzzle","threadId":"7415","inReplyTo":"m23b3qts11.fsf@ziti.fhcrc.org","subject":"[PATCH] git-svn: avoid respewing similar error messages for missing paths","fromName":"Eric Wong","fromEmail":"normalperson@yhbt.net","sentAt":"2007-03-31T00:54:48Z","receivedAt":"2007-03-31T00:54:48Z","isPatch":true,"sender":{"key":"e@80x24.org","avatar":null},"body":"We ignore errors if the path we're tracking did not exist for\na particular revision range, but we still print out warnings\ntelling the user about that.\n\nAs pointed out by Seth Falcon, this amounts to a lot of warnings\nthat could confuse and worry users. I'm not entirely comfortable\ncompletely silencing the warnings, but showing one warning per\npath that we track should be reasonable.\n\nSigned-off-by: Eric Wong \n---\n git-svn.perl | 16 +++++++++++-----\n 1 files changed, 11 insertions(+), 5 deletions(-)\n\ndiff --git a/git-svn.perl b/git-svn.perl\nindex adc976c..d307d43 100755\n--- a/git-svn.perl\n+++ b/git-svn.perl\n@@ -2843,8 +2843,7 @@ package Git::SVN::Ra;\n use vars qw/@ISA $config_dir $_log_window_size/;\n use strict;\n use warnings;\n-my ($can_do_switch);\n-my $RA;\n+my ($can_do_switch, %ignored_err, $RA);\n \n BEGIN {\n \t# enforce temporary pool usage for some simple functions\n@@ -3216,9 +3215,16 @@ sub skip_unknown_revs {\n \t# 175007 - http(s):// (this repo required authorization, too...)\n \t# More codes may be discovered later...\n \tif ($errno == 175007 || $errno == 175002 || $errno == 160013) {\n-\t\twarn \"W: Ignoring error from SVN, path probably \",\n-\t\t \"does not exist: ($errno): \",\n-\t\t $err->expanded_message,\"\\n\";\n+\t\tmy $err_key = $err->expanded_message;\n+\t\t# revision numbers change every time, filter them out\n+\t\t$err_key =~ s/\\d+/\\0/g;\n+\t\t$err_key = \"$errno\\0$err_key\";\n+\t\tunless ($ignored_err{$err_key}) {\n+\t\t\twarn \"W: Ignoring error from SVN, path probably \",\n+\t\t\t \"does not exist: ($errno): \",\n+\t\t\t $err->expanded_message,\"\\n\";\n+\t\t\t$ignored_err{$err_key} = 1;\n+\t\t}\n \t\treturn;\n \t}\n \tdie \"Error from SVN, ($errno): \", $err->expanded_message,\"\\n\";\n-- \nEric Wong\n"},{"id":"38478","messageId":"m2lkhb6pbm.fsf@gmail.com","threadId":"7415","inReplyTo":"20070331005448.GA6948@muzzle","subject":"Re: [PATCH] git-svn: avoid respewing similar error messages for missing paths","fromName":"Seth Falcon","fromEmail":"sethfalcon@gmail.com","sentAt":"2007-04-02T10:30:37Z","receivedAt":"2007-04-02T10:30:37Z","isPatch":true,"sender":{"key":"sethfalcon@gmail.com","avatar":"https://gravatar.com/avatar/fd62fe73d3013b12fce71d5269ec52eaca3c4cee479efc6eea3603d6f1d8bf8d?d=mp&s=160"},"body":"Hi Eric,\n\nEric Wong writes:\n\n> We ignore errors if the path we're tracking did not exist for\n> a particular revision range, but we still print out warnings\n> telling the user about that.\n>\n> As pointed out by Seth Falcon, this amounts to a lot of warnings\n> that could confuse and worry users. I'm not entirely comfortable\n> completely silencing the warnings, but showing one warning per\n> path that we track should be reasonable.\n\nThanks, this is much quieter.\n\nI have the impression that the initial fetch phase is taking longer\nthan it used to. This could easily be due to network and server load\nrelated issues, but I wonder if something in the parent following /\npath searching is different. The first git svn fetch seems to do\nnothing for a very long time (except emit a few of the warnings) and\nthen once it starts actually adding commits it goes quite fast. This\nisn't really a problem as the initial fetch is a fairly rare\noperation, but I wondered if this made sense based on recent git-svn\nchanges...\n\n\n+ seth\n"},{"id":"38488","messageId":"20070402184652.GA29646@hand.yhbt.net","threadId":"7415","inReplyTo":"m2lkhb6pbm.fsf@gmail.com","subject":"Re: [PATCH] git-svn: avoid respewing similar error messages for missing paths","fromName":"Eric Wong","fromEmail":"normalperson@yhbt.net","sentAt":"2007-04-02T18:46:53Z","receivedAt":"2007-04-02T18:46:53Z","isPatch":true,"sender":{"key":"e@80x24.org","avatar":null},"body":"Seth Falcon wrote:\n> I have the impression that the initial fetch phase is taking longer\n> than it used to. This could easily be due to network and server load\n> related issues, but I wonder if something in the parent following /\n> path searching is different. The first git svn fetch seems to do\n> nothing for a very long time (except emit a few of the warnings) and\n> then once it starts actually adding commits it goes quite fast. This\n> isn't really a problem as the initial fetch is a fairly rare\n> operation, but I wondered if this made sense based on recent git-svn\n> changes...\n\nDoes using --log-window-size=1000 speed things back up?\n\nI decreased it to 100 to reduce memory usage (but exposed it as an\noption).\n\n-- \nEric Wong\n"}]}