git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: git cherry improvements suggestions

From
Michael J Gruber <git@drmicha.warpmail.net>
Date
Mar 23, 2011, 14:44 UTC
Message-ID
<4D8A0741.9000506@drmicha.warpmail.net>
In-Reply-To
<AANLkTin7Du5RYt946hNjP-y53puNykebCjiKk5Ju_igr@mail.gmail.com>
Piotr Krukowiecki venit, vidit, dixit 23.03.2011 15:43:
Show 43 quoted lines
> On Wed, Mar 23, 2011 at 2:33 PM, Michael J Gruber
> <git@drmicha.warpmail.net> wrote:
>> Piotr Krukowiecki venit, vidit, dixit 23.03.2011 14:23:
>>> Just some stats:
>>>
>>>    git log --cherry-mark --left-right --oneline --date-order branch...trunk
>>>
>>> lists 1004 commits, takes about 20s and memory peaks to about 670MB
>>> twice during the run (I'm on linux with AMD Phenom II X4 945)
>>>
>>> With limit it prints X last commits (the limiting seems to take place after all
>>> work, on the output list only).
>>>
>>> branch..trunk is 551 commits, the other way is 453 commits.
>>> 710 commits are found to be "=", 98 "<", 196 ">".
>>>
>>> Note, I'm not saying it's too slow, or that it's working incorrectly, I'm just
>>> giving real-life stats if anyone was interested.
>>> I suspect such checks won't be done frequently.
>>
>> You don't need to say it's slow - I've said so already :(
>>
>> http://permalink.gmane.org/gmane.comp.version-control.git/169725
> 
> In the link above:
>   git cherry A B: 0.4s
>   git rev-list --cherry A...B: 1.7s
> 
> So rev-list is 4.25x slower.
> 
> In my case it's only 1.23x slower:
> 
>    $ time git rev-list --cherry branch...trunk  > /tmp/rev-list
>    real	0m18.627s
>    user	0m17.710s
>    sys	0m0.900s
> 
>    $ time git cherry  branch trunk  > /tmp/cherry
>    real	0m15.345s
>    user	0m14.310s
>    sys	0m1.020s
> 
> 

How's that with > /dev/null (or with --count for rev-list)? Also, how many merge bases do you have:

git merge-base --all branch trunk | wc -l

Thanks! Michael

Previous: Piotr KrukowieckiNext: Piotr Krukowiecki
Message 6 of 9 in “git cherry improvements suggestions”
  1. Piotr KrukowieckiMar 23, 2011
  2. Michael J GruberMar 23, 2011
  3. Piotr KrukowieckiMar 23, 2011
  4. Michael J GruberMar 23, 2011
  5. Piotr KrukowieckiMar 23, 2011
  6. Michael J GruberMar 23, 2011
  7. Piotr KrukowieckiMar 23, 2011
  8. Piotr KrukowieckiMar 24, 2011
  9. Michael J GruberMar 24, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.