On Wed, 2011-11-02 at 19:15 +0100, Jakub Hrozek wrote:
On Wed, Nov 02, 2011 at 01:53:02PM -0400, Stephen Gallagher wrote:
> On Wed, 2011-11-02 at 13:49 -0400, Stephen Gallagher wrote:
> > On Tue, 2011-11-01 at 08:50 +0100, Jan Zelený wrote:
> > > > On Mon, Oct 31, 2011 at 04:37:03PM -0400, Stephen Gallagher wrote:
> > > > > SSSD 1.6.2 has a major performance issue due to the addition of
the
> > > > > alias support. We added a new attribute to the lookup filter,
aliasName.
> > > > > Unfortunately, we forgot to add an index to it and it's
causing a severe
> > > > > bottleneck in cache lookups.
> > > >
> > > > Ack
> > >
> > > Nack,
> > > - tmp_ctx should be allocated on NULL
> > > - please use finish_upgrade() instead of duplicating the code
> >
> >
> > Thanks for the review. New patch attached.
>
> Attached the wrong patch. Correct one attached now.
Jan's comments were taken into account, patch still works fine.
Ack.
Pushed to master.