Re: [PATCH 0/4] check_everything_connected replacement

2013-04-01 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy writes: > The last use of check_everything_connected after this series is > fetch.c:quickfetch(), which I think is unnecessary. It can only catch > errors if we have incomplete object islands in repo, which could > happen before this series. A typical source of object islan

[PATCH 0/4] check_everything_connected replacement

2013-03-31 Thread Nguyễn Thái Ngọc Duy
My investigation in lowering connectivity check cost in git-clone [1] led me to try 'index-pack --strict' code. Without calling fsck_object(), --strict seems to be cheaper than check_everything_connected() while accomplishing the same thing (imo). The first patch is a bug fix running git-clone --d