On Jan 10, 2008 2:07 AM, C. Michael Pilato <cmpilato_at_collab.net> wrote:
> I was trying to help Hyrum out by looking in the log-tests 18 failure.
> This was my first time seeing 'svn log -g' output in a while, and it
> took a while to understand what I was reading. (Karl assisted greatly
> by changing the text used to report merged-via info.) Anyway, I
> noticed an absurd amount of duplicated data in the output stream of
> 'svn log -g' as run on the test data for this test. (See
> http://paste.lisp.org/display/53925 to see what I see.) It's a hard
> problem, trying to take naturally tree-like, nested data and display
> it in a flat output. But here's a half-baked idea for doing so that I
> wanted to bounce off of folks.
>
> First, a nasty ASCII branch diagram which is supposed to correspond to
> my later sample output:
>
> --|-----|-----------------|--------------
> / 2 3 6 \
> / \
> |-----------------|-----|-----------------|-----|-----|------> /trunk
> 1 4 \ 5 8 /9 / 10
> \ / /
> --|---------------------------
> 5 \ /
> \ /
> ----------
> 7
>
> Disclaimer: I'm quite sleepy, so try not to get bogged down in
> specific revision numbers used in my sample output. I probably goofed
> up in a few spots.
>
> Now, today's 'svn log -g' code will, I believe, transmit the following
> stream of revisions (shown nested based on log entry's has-children bit)
> across the wire for a log of /trunk in the diagram above. (The nesting
> indicates that some revisions are part of the history of the requested
> object because they were merged into that history as part of some
> natural revision of the object in question.)
>
> r10
> r5
> r4
> r1
> --
> r9
> r6
> r3
> r2
> r1
> --
> r7
> r5
> r4
> r1
> --
> r8
> r5
> r4
> r1
>
> I was thinking tonight about all those duplicated revisions, and
> wondering if they couldn't be flattened out by the client, with dupes
> removed and revisions kept in order. The proposed output would be
> something like:
>
> ------------------------------------------------------------------------
> r10 | ...
> Changed paths:
> ...
> Lineage: natural
>
> ------------------------------------------------------------------------
> r9 | ...
> Changed paths:
> ...
> Lineage: natural
>
> ------------------------------------------------------------------------
> r8 | ...
> Changed paths:
> ...
> Lineage: natural
>
> ------------------------------------------------------------------------
> r7 | ...
> Changed paths:
> ...
> Lineage: merged via r9
>
> ------------------------------------------------------------------------
> r6 | ...
> Changed paths:
> ...
> Lineage: merged via r9
>
> ------------------------------------------------------------------------
> r5 | ...
> Changed paths:
> ...
> Lineage: natural, merged via r10, r9
>
> ------------------------------------------------------------------------
> r4 | ...
> Changed paths:
> ...
> Lineage: natural, merged via r10, r9
>
> ------------------------------------------------------------------------
> r3 | ...
> Changed paths:
> ...
> Lineage: merged via r9
>
> ------------------------------------------------------------------------
> r2 | ...
> Changed paths:
> ...
> Lineage: merged via r9
>
> ------------------------------------------------------------------------
> r1 | ...
> Changed paths:
> ...
> Lineage: natural, merged via r10, r9
>
> ------------------------------------------------------------------------
>
> It would require the client to keep a cache of the merge tree and the
> not-yet-printed revision metadata. Knowing that our client was
> responsible for maintaining such a thing, though, could possibly allow
> us to teach the server to stop duplicating that data on the wire.
> (Part of the contract is that in -g mode, 'svn log' sends real
> metadata only once per revision, only revision numbers for
> duplicates.)
>
> Oh -- and this would only apply to 'svn log -g' when not in --xml output mode.
>
> Comments? Questions? Tomatoes?
Hmm. I don't really like the current 'svn log -g' output either. But
I'm not sure that your suggestion is very useful.
Frankly, we need to ask the question: what is the point of 'svn log
-g'? My impression was that it was roughly "Instead of copying log
messages from merged revisions into the merging revision's log message
like svnmerge.py and svk do by default, get them later at 'svn log -g'
time". And in that case, the thing the user is trying to discover is
"OK, I see that r9 is a merging revision, but what are the changes
that it is actually merging?" And in that case, having the answer be
immediate is much more useful than waiting until the merged revisions
happen to show up in numerical order.
I think we could do a lot better about actually printing the messages,
though. I'm still somewhat partial to indenting the entries, although
that doesn't work well past the first two or three levels; or we could
use a different separator than the '-----' line (eg, "=== revisions
merged in r9 ====" and "=== end of revisions merged in r9 ===".
--dave
--
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-01-10 16:27:50 CET