Re: [PATCH 10/10] fast-export: add --always-show-modify-after-rename

2018-11-13 Thread Jeff King
On Tue, Nov 13, 2018 at 09:10:36AM -0800, Elijah Newren wrote: > > I am looking at this problem as "how do you answer question X in a > > repository". And I think you are looking at as "I am receiving a > > fast-export stream, and I need to answer question X on the fly". > > > > And that would

Re: [PATCH 10/10] fast-export: add --always-show-modify-after-rename

2018-11-13 Thread Elijah Newren
On Tue, Nov 13, 2018 at 6:45 AM Jeff King wrote: > It is an expensive log command, but it's the same expense as running > fast-export, no? And I think maybe that is the disconnect. I would expect an expensive log command to generally be the same expense as running fast-export, yes. But I would

Re: [PATCH 10/10] fast-export: add --always-show-modify-after-rename

2018-11-13 Thread Jeff King
On Mon, Nov 12, 2018 at 10:08:10AM -0800, Elijah Newren wrote: > > I would do: > > > >git log --raw $( > > git cat-file --batch-check='%(objectsize:disk) %(objectname)' > > --batch-all-objects | > > sort -rn | head -3 | > > awk '{print "--find-object=" $2 }' > >) > > > >

Re: [PATCH 10/10] fast-export: add --always-show-modify-after-rename

2018-11-12 Thread Elijah Newren
On Mon, Nov 12, 2018 at 4:58 AM Jeff King wrote: > On Sun, Nov 11, 2018 at 12:42:58AM -0800, Elijah Newren wrote: > > Maybe I don't understand what you're trying to accomplish. I was > thinking specifically of your "cat-file can tell you the large objects, > but you don't know their

Re: [PATCH 10/10] fast-export: add --always-show-modify-after-rename

2018-11-12 Thread Jeff King
On Sun, Nov 11, 2018 at 12:42:58AM -0800, Elijah Newren wrote: > > > fast-export output is traditionally used as an input to a fast-import > > > program, but it is also useful to help gather statistics about the > > > history of a repository (particularly when --no-data is also passed). > > > For

Re: [PATCH 10/10] fast-export: add --always-show-modify-after-rename

2018-11-11 Thread Elijah Newren
On Sat, Nov 10, 2018 at 11:23 PM Jeff King wrote: > > On Sat, Nov 10, 2018 at 10:23:12PM -0800, Elijah Newren wrote: > > > fast-export output is traditionally used as an input to a fast-import > > program, but it is also useful to help gather statistics about the > > history of a repository

Re: [PATCH 10/10] fast-export: add --always-show-modify-after-rename

2018-11-10 Thread Jeff King
On Sat, Nov 10, 2018 at 10:23:12PM -0800, Elijah Newren wrote: > fast-export output is traditionally used as an input to a fast-import > program, but it is also useful to help gather statistics about the > history of a repository (particularly when --no-data is also passed). > For example, two of

[PATCH 10/10] fast-export: add --always-show-modify-after-rename

2018-11-10 Thread Elijah Newren
fast-export output is traditionally used as an input to a fast-import program, but it is also useful to help gather statistics about the history of a repository (particularly when --no-data is also passed). For example, two of the types of information we may want to collect could include: 1)