Re: [PATCH]log and blame command should log the revision range in operational log

On Mon, 26 Feb 2007, Kamesh Jayachandran wrote:

>
> Dan,
>
> >Why do we call the action "log-all" instead of just "log"? It'd also
> >be nice if we had a better name for "log-partial", if we do want to
> >use a different name to indicate that it's a log of multiple paths
> >instead of just 1. I'd personally be in favor of calling them both
> >"log", for simplicity's sake -- we now have the paths listed, which
> >can be used to differentiate between single-path and multi-path
> >invocations of 'log'.
> >
> >I'm attaching a patch which implements my thoughts -- let me know
> >what'cha think.
> >
> >...
> >
>
> Your patch looks precise and neat.
>
> My +1.

Thanks Kamesh! I was just looking at the "ops" log generated by
log_tests.py over ra_serf, and noticed that we're logging r0 quite
often (usually for the end revision, and sometimes for both
revisions). Looking at the code, it looks like this is the value
we're probably using for the 'log' operation. It just seemed a little
strange to me when looking at the operational logging output, so I
wanted to double-check that this is a) expected and b) desirable.

[27/Feb/2007:13:35:36 -0800] jrandom log '/' r9:0
[27/Feb/2007:13:35:44 -0800] jrandom log '/' r0:0
[27/Feb/2007:13:35:48 -0800] jrandom log '/A/D/H' r2:2
[27/Feb/2007:13:35:52 -0800] jrandom log '/' r1:0
[27/Feb/2007:13:36:13 -0800] jrandom log '/' r9:9
[27/Feb/2007:13:36:13 -0800] jrandom log '/' r9:9
[27/Feb/2007:13:36:13 -0800] jrandom log '/' r9:9
[27/Feb/2007:13:36:13 -0800] jrandom log '/' r8:8
[27/Feb/2007:13:36:35 -0800] jrandom log '/A/D/G, /A/D/H' r9:0
[27/Feb/2007:13:36:55 -0800] jrandom log '/A/B/E/beta' r8:0
[27/Feb/2007:13:37:14 -0800] jrandom log '/A/B/E/alpha' r8:0
[27/Feb/2007:13:37:25 -0800] jrandom log '/A/mu2' r6:0
[27/Feb/2007:13:37:25 -0800] jrandom log '/A/mu2' r6:0
[27/Feb/2007:13:37:25 -0800] jrandom log '/A/mu2' r3:3
[27/Feb/2007:13:37:25 -0800] jrandom log '/A/mu2' r3:3
[27/Feb/2007:13:37:25 -0800] jrandom log '/A/mu' r2:2
[27/Feb/2007:13:37:25 -0800] jrandom log '/A/mu' r2:2
[27/Feb/2007:13:37:26 -0800] jrandom log '/' r1:0
[27/Feb/2007:13:37:29 -0800] jrandom log '/' r1:1
[27/Feb/2007:13:37:29 -0800] jrandom log '/' r1:1
[27/Feb/2007:13:37:48 -0800] jrandom log '/' r9:0
[27/Feb/2007:13:37:48 -0800] jrandom log '/A/B' r9:0
[27/Feb/2007:13:37:48 -0800] jrandom log '/A/B' r2:9
[27/Feb/2007:13:37:48 -0800] jrandom log '/A/B' r1:1
[27/Feb/2007:13:38:08 -0800] jrandom log '/A/B/E/beta' r9:0
[27/Feb/2007:13:38:09 -0800] jrandom log '/A/B/E/beta' r1:0