Re: [PATCH 0/5] Use watchman to reduce index refresh time

2015-11-20 Thread Christian Couder
On Tue, Nov 10, 2015 at 10:04 PM, David Turner wrote: > On Mon, 2015-11-09 at 21:06 +0100, Christian Couder wrote: >> Using David's series I get worse results than all of the above but I >> guess it's because his series is based on an ancient git version >> (v2.0.0-rc0).

Re: [PATCH 0/5] Use watchman to reduce index refresh time

2015-11-10 Thread David Turner
On Mon, 2015-11-09 at 21:06 +0100, Christian Couder wrote: > Using David's series I get worse results than all of the above but I > guess it's because his series is based on an ancient git version > (v2.0.0-rc0). My more-recent series is on top of 2.4, but (for webkit): mine: 0m0.206s duy's:

Re: [PATCH 0/5] Use watchman to reduce index refresh time

2015-11-09 Thread Christian Couder
On Tue, Nov 3, 2015 at 10:21 AM, Duy Nguyen wrote: > On Mon, Nov 2, 2015 at 8:23 PM, Duy Nguyen wrote: >> On Mon, Nov 2, 2015 at 3:54 PM, Paolo Ciarrocchi >> wrote: >>> On Sun, Nov 1, 2015 at 2:55 PM, Nguyễn Thái Ngọc Duy

Re: [PATCH 0/5] Use watchman to reduce index refresh time

2015-11-03 Thread Duy Nguyen
On Mon, Nov 2, 2015 at 8:23 PM, Duy Nguyen wrote: > On Mon, Nov 2, 2015 at 3:54 PM, Paolo Ciarrocchi > wrote: >> On Sun, Nov 1, 2015 at 2:55 PM, Nguyễn Thái Ngọc Duy >> wrote: >> >> Hi Duy, >> >>> This series builds on top of

Re: [PATCH 0/5] Use watchman to reduce index refresh time

2015-11-03 Thread Paolo Ciarrocchi
On Tue, Nov 3, 2015 at 10:21 AM, Duy Nguyen wrote: >> It was from last year. I may have measured it but because I didn't >> save it in the commit message, it was lost anyway. Installing watchman >> and measuring with webkit.git soon.. > > Test repo: webkit.git with 104665

Re: [PATCH 0/5] Use watchman to reduce index refresh time

2015-11-02 Thread Paolo Ciarrocchi
On Sun, Nov 1, 2015 at 2:55 PM, Nguyễn Thái Ngọc Duy wrote: Hi Duy, > This series builds on top of the index-helper series I just sent and > uses watchman to keep track of file changes in order to avoid lstat() > at refresh time. The series can also be found at [1] > > When I

Re: [PATCH 0/5] Use watchman to reduce index refresh time

2015-11-02 Thread Duy Nguyen
On Mon, Nov 2, 2015 at 3:54 PM, Paolo Ciarrocchi wrote: > On Sun, Nov 1, 2015 at 2:55 PM, Nguyễn Thái Ngọc Duy > wrote: > > Hi Duy, > >> This series builds on top of the index-helper series I just sent and >> uses watchman to keep track of file

[PATCH 0/5] Use watchman to reduce index refresh time

2015-11-01 Thread Nguyễn Thái Ngọc Duy
This series builds on top of the index-helper series I just sent and uses watchman to keep track of file changes in order to avoid lstat() at refresh time. The series can also be found at [1] When I started this work, watchman did not support Windows yet. It does now, even if still experimental