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

Re: [BUG] resolved deltas

From
Petr Stodulka <pstodulk@redhat.com>
Date
Aug 21, 2014, 18:25 UTC
Message-ID
<53F639A3.80805@redhat.com>
In-Reply-To
<53F5D98F.4040700@redhat.com>
Show 7 quoted lines
> <snip>
> Bug is reprodusible since git version 1.8.3.1 (may earlier 1.8.xx, but 
> I don't test it) to actual upstream version.
> This problem "doesn't exists" in version 1.7.xx - or more precisely is 
> not reproducible. "May" this is reproducible
> since commit "7218a215" - in this commit was added assert in file 
> "builtin/index-pack.c" (actual line is 918), but I didn't test this.
Ok so this is reproducible since this commit because of assert().
Show 13 quoted lines
> Here I am lost. I don't know really what I can do next here, because I 
> don't understand some ideas in code. e.g. searching of child - 
> functions find_delta(), find_delta_children(). Calculation on line 618:
> ----
>     int next = (first+last) / 2;
> ----
> I still don't understand. I didn't find description of this searching 
> algorithm in tech. documentation but I didn't read all yet. However I 
> think that source of problems could be somewhere in these two 
> functions. When child is found, its real_type is set to parent's type 
> in function resolve_delta() on the line 865 and then lasts wait for 
> failure. I don't think that problem is in repository itself [1], but 
> it is possible.

I read history of commits and my idea seems to be incorrect. It seems more like some error in repository itself. But I'd rather get opinion from someone who knows this code and ideas better.

Regards, Petr

> [0] https://bugzilla.redhat.com/show_bug.cgi?id=1099919
> [1] git clone https://code.google.com/p/mapsforge/ mapsforge.git
Previous: Petr StodulkaNext: Martin von Gagern
Message 2 of 30 in “[BUG] resolved deltas”
  1. Petr StodulkaAug 21, 2014
  2. Petr StodulkaAug 21, 2014
  3. Martin von GagernAug 22, 2014
  4. René ScharfeAug 23, 2014
  5. Jeff KingAug 23, 2014
  6. Jeff KingAug 23, 2014
  7. Jeff KingAug 23, 2014
  8. René ScharfeAug 25, 2014
  9. Jeff KingAug 28, 2014
  10. Jeff KingAug 28, 2014
  11. Jeff KingAug 28, 2014
  12. Jeff KingAug 28, 2014
  13. Junio C HamanoAug 28, 2014
  14. Jeff KingAug 29, 2014
  15. 1/2 index-pack: fix race condition with duplicate basesJeff King, Aug 29, 2014
  16. 2/2 index-pack: handle duplicate base objects gracefullyJeff King, Aug 29, 2014
  17. Junio C HamanoAug 29, 2014
  18. Jeff KingAug 29, 2014
  19. Shawn PearceAug 30, 2014
  20. Jeff KingAug 30, 2014
  21. René ScharfeAug 30, 2014
  22. Jeff KingAug 31, 2014
  23. René ScharfeAug 31, 2014
  24. Shawn PearceAug 31, 2014
  25. Jeff KingAug 31, 2014
  26. Junio C HamanoAug 31, 2014
  27. 3/2 t5309: mark delta-cycle failover tests as passingJeff King, Aug 30, 2014
  28. Jeff KingAug 31, 2014
  29. Junio C HamanoSep 2, 2014
  30. Shawn PearceAug 25, 2014

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.