Re: [egit-dev] staging view and indexing quirks

[Robin Rosenberg <robin.rosenberg@xxxxxxxxxx>]

I've added a bug on the re-indexing issue: https://bugs.eclipse.org/bugs/show_bug.cgi?id=373077

Markus Duft skrev 2012-03-09 10.01:
> On 03/08/2012 06:33 PM, Marc-Andre Laperle wrote:
> > On the topic of indexing quirks, I also noticed that when building projects, a lot indexing jobs can be started for the same repo. Sometimes the extra ones get cancelled but sometimes they don't and it then takes forever for all of them to complete. Anyone else seeing that?
>
> Also (you experts, please answer :D), is there a way to get index change event in a way that a list of changed paths is included? this would make it possible to just calculate index diff cache changes only for the changed paths....

In the current implementation: no. The original use of the index event service was to detect that an
external change happened and I didn't want to cache old index. How could I e.g. know that a path disappeared.

I think the basic problem here is that the re-indexing jobs get run and canceled too often or simply work
too slowly. You could get more details and do more advances stuff with more detailed events, but I think
we should check that the re-index operation is as fast as it could possibly be without magic first.


-- robin

> regards,
> Markus
>
> > Marc-Andre
> >
> > On 08/03/2012 11:12 AM, Markus Duft wrote:
> > > Hey!
> > >
> > > I have made a little change [1] to the staging view. Its still in a "pre-alpha" stage, so don't expect everything to be perfect. i just need some review and input whether i'm on the right track.
> > >
> > > the story behind the change is, that in our workspace (approx 600 projects with approx 6 million line of code) the re-indexing of the repo takes ages (speak ~1 minute on a fast machine). thus we want to as far as possible avoid re-indexing. the staging view now has it's own indexing job, which runs independently of the indexDiffCache update jobs (am i wrong here?). i want to get rid of this, so we can (finally) start using the staging view.
> > >
> > > [1] https://git.eclipse.org/r/#/c/5299/
> > >
> > > regards, and thanks for the input in advance :D
> > > Markus
> > > _______________________________________________
> > > egit-dev mailing list
> > > egit-dev@xxxxxxxxxxx
> > > https://dev.eclipse.org/mailman/listinfo/egit-dev
> >
> > _______________________________________________
> > egit-dev mailing list
> > egit-dev@xxxxxxxxxxx
> > https://dev.eclipse.org/mailman/listinfo/egit-dev
> _______________________________________________
> egit-dev mailing list
> egit-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/egit-dev