First writings on DsGetNCChanges call

[Anatoliy Atanasov]

I fixed it, thanks.

I also fixed the usage of dsdb_attribute_ldb_to_drsuapi function so now it returns properly.

There is still the problem with the call to dsdb_syntax_FOOBAR_ldb_to_drsuapi in source4/dsdb/schema/schema_syntax.c:735 (commit 6488afaa). The DSSYNC test passes when I remove that call.

Anatoliy

> -----Original Message-----
> From: tridge at samba.org [mailto:tridge at samba.org]
> Sent: Wednesday, August 26, 2009 05:07
> To: Anatoliy Atanasov
> Cc: abartlet at samba.org; metze at samba.org; samba-technical at lists.samba.org
> Subject: Re: First writings on DsGetNCChanges call
> 
> Hi Anatoliy,
> 
>  > I uploaded the work-in-progress branch here:
> http://repo.or.cz/w/Samba/aatanasov.git?a=shortlog;h=refs/heads/samba_drsu
> api
> 
> thanks for posting this!
> 
> It looks like in your dcesrv_drsuapi_DsGetNCChanges() call you do a
> full search of the directory every time, pulling all records?
> 
> Just using a search of (uSNChanged>=N) would be a big improvement in
> terms of memory consumption. It will still be slow in the backend, as
> the tdb backend doesn't actually use the indexes for a >= search, but
> at least the DRS server wouldn't have to parse all the records.
> 
> Cheers, Tridge