defining the new idmap subsystem

[simo]

On Mon, 2006-10-02 at 17:52 -0700, Jeremy Allison wrote:
> On Mon, Oct 02, 2006 at 08:45:27PM -0400, simo wrote:
> > I have been working on a first draft on how I would like to change the
> > current IDmap subsystem.
> > 
> > The design and API described in it are the result of the discussions we
> > had at the CIFS conference as well on IRC.
> > 
> > The internal set of interfaces is not yet defined but will resemble the
> > one used in winbindd.
> > 
> > Comments, corrections or critics are very welcome,
> > I will post new revisions of the document at the same URL as I dig into
> > the topic, this week.
> > 
> > This is it: http://samba.org/~idra/samba3_newidmap.pdf
> 
> Sorry, I *hate* the flags : FLAG_QUERY_ONLY, FLAG_ALLOCATE.
> 
> Please make these separate calls in the interface - expose
> the allocate functionality to the caller.

The problem is that this will duplicate all query calls, it's doable,
but I wonder if it is really worth it.

> The problem with the existing idmap is that we confuse query
> and allocate - I don't want to see that again.

So you would like to see something like splitting
idmap_uid_to_sid() into idmap_uid_to_sid_query() and
idmap_uid_to_sid_allocate() ?

I think the code will be much uglier but if most want that, so be it.
But I'd like some more people complaining about that, before changing.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer
email: idra at samba.org
http://samba.org