[PATCH] faster initial commit due to buffered logging

Ian Clatworthy ian.clatworthy at internode.on.net
Wed May 30 00:33:32 BST 2007


Robert Collins wrote:
> On Mon, 2007-05-28 at 21:59 +1000, Ian Clatworthy wrote:
>   
>
> Logging stuff to ~/.bzr.log should be done where we think it will help
> debug problems. Specifically in this case, the actions are already
> logged to the console, so there is no need to also log them to
> ~/.bzr.log. I suggested switch to stdout because that will:
>  - remove the ~/.bzr.log output
>  - make commit more amenable to redirection
>  - put what is arguably the expected output of commit - the things its
>    doing - into stdout.
>
>
>   
I'm good with this - I just doesn't feel qualified to make a policy 
decision re what should and shouldn't go in ~/.bzr.log. Great to have 
you back!

I'll make sure I comment it clearly so someone doesn't come along later 
and put things back and 'fix' commit to update the log file.
>
> I like making the progress bar useful, but I dont think we need the
> complexity of your explicit buffering; rather lets just not write to
> ~/.bzr.log per-file-added. Also, the right place to buffer ~/.bzr.log,
> if we choose to do that, would be in the file itself: that way you wont
> change the ordering of things output to the file from different places
> in bzrlib, which would be a semantic change with confusing side effects.
>   

An excellent point. For consistency, "per file-added" needs to be 
"per-file" though, i.e. all the messages coming out of the CommitReporter.

Ian C.



More information about the bazaar mailing list