[Freeipa-devel] [PATCH 0/3] sssd: Rework sysdb to use tevent_req async style

[Simo Sorce]

On Fri, 2009-07-03 at 14:54 +0200, Sumit Bose wrote:
> 
> I have tested them together and have not found any issues. Therefore
> 
> ACK
> ACK
> ACK

Ok, pushed to master.

> > I have not yet converted the responder and the sysdb_getXXyyy calls
> > (except for minor internal adjustments), not sure if it is worth
> > changing that interface yet.
> > 
> 
> I think it would make sense to have only one interface. Shall I take
> the
> pam responder?

Do you have in mind any change beyond just how you fetch information
about users ?

> > I may convert the backends interfaae to use directly a tevent_req
> style
> > later, but preferred to go in steps. I might propose such a patch
> > together with the ldap driver.
> > 
> 
> +1
> This will underline that the backends have to be asynchronous and have
> to/should use tevent to achieve this.

This is the idea.

> Thanks for the redesign. I'm looking forward to the new LDAP backend.

Yeah working on that :-)

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York