Re: [PATCH v4 4/6] revision: implement sparse algorithm
- From
Derrick Stolee <stolee@gmail.com>
- Date
- Dec 17, 2018, 14:20 UTC
- Message-ID
- <867aa5c3-60e0-2467-795a-40aac58f306b@gmail.com>
- In-Reply-To
- <87efaj1y77.fsf@evledraar.gmail.com>
On 12/14/2018 6:32 PM, Ævar Arnfjörð Bjarmason wrote:
Show 13 quoted lines
> On Fri, Dec 14 2018, Derrick Stolee via GitGitGadget wrote: > >> Despite these potential drawbacks, the benefits of the algorithm >> are clear. By adding a counter to 'add_children_by_path' and >> 'mark_tree_contents_uninteresting', I measured the number of >> parsed trees for the two algorithms in a variety of repos. > We spend a long time printing those out before we ever get to > "Enumerating objects". > > Which was where I was trying to test this, i.e. is this a lot of work we > perform before we print out the progress bar, and regardless of this > optimization should have other progress output there, so we can see this > time we're spending on this?
It is true that part of the problem is that a 'git push' will sit for a while without presenting any feedback until this part of the algorithm is complete. The current series intends to significantly reduce this time.
As for adding progress to this step, I'm open to it. It can be done as a sequel series.
What would we use to describe this section? "Enumerating remote objects"?
Thanks, -Stolee