#47963 RFE - memberOf - add option to skip nested group lookups during delete operations
Closed: wontfix None Opened 9 years ago by mreynolds.

The recursive nested group lookups performed during a group delete operation can take a very long time to complete if there are very large static groups(groups with with over 10K members). If there are no nested groups, then it would be nice to have an option to skip the nested group check, which would significantly improve delete performance.


I think you need to free this skip_nested:
{{{
if ((skip_nested = slapi_entry_attr_get_charptr(e, MEMBEROF_SKIP_NESTED_ATTR))){
}}}
and this one
{{{
skip_nested = slapi_entry_attr_get_charptr(e, MEMBEROF_SKIP_NESTED_ATTR);
}}}

Replying to [comment:2 rmeggins]:

I think you need to free this skip_nested:
{{{
if ((skip_nested = slapi_entry_attr_get_charptr(e, MEMBEROF_SKIP_NESTED_ATTR))){
}}}
and this one
{{{
skip_nested = slapi_entry_attr_get_charptr(e, MEMBEROF_SKIP_NESTED_ATTR);
}}}

Yeah, nice catch. New patch attached.

c973e71..b01cf4d master -> master
commit b01cf4d
Author: Mark Reynolds mreynolds@redhat.com
Date: Mon Nov 24 16:58:57 2014 -0500

44f84b3..ecd34a5 389-ds-base-1.3.3 -> 389-ds-base-1.3.3
commit ecd34a5

9c44c63..9cce9c4 389-ds-base-1.3.2 -> 389-ds-base-1.3.2
commit 9cce9c4

cd06271..250fcdb 389-ds-base-1.3.1 -> 389-ds-base-1.3.1
commit 250fcdbb463d2f4597a61ef1e364f71fa01ef1be

0c47dfb..ec0b121 389-ds-base-1.2.11 -> 389-ds-base-1.2.11
commit ec0b121

c5cc125..d8e8119 master -> master
commit d8e8119
Author: Mark Reynolds mreynolds@redhat.com
Date: Wed Nov 26 16:57:05 2014 -0500

ecd34a5..ae06f96 389-ds-base-1.3.3 -> 389-ds-base-1.3.3
commit ae06f96

9cce9c4..919f30c 389-ds-base-1.3.2 -> 389-ds-base-1.3.2
commit 919f30c

250fcdb..664b4f1 389-ds-base-1.3.1 -> 389-ds-base-1.3.1
commit 664b4f126b356854806762f880274b5531c83fb1

ec0b121..116dcbd 389-ds-base-1.2.11 -> 389-ds-base-1.2.11
commit 116dcbd

I have done some tests and I have found some problems with this config option. The plugin seems not to be working.

Re opening this bug.

Will give further details in the BZ with my testcase.

4681fab..248b471 master -> master
commit 248b471
Author: Mark Reynolds mreynolds@redhat.com
Date: Tue Feb 3 12:52:40 2015 -0500

d65945e..2f24d28 389-ds-base-1.3.3 -> 389-ds-base-1.3.3
commit 2f24d28

a204469..70d5fe6 389-ds-base-1.3.2 -> 389-ds-base-1.3.2
commit 70d5fe6

bf55d5a..ddcab8a 389-ds-base-1.3.1 -> 389-ds-base-1.3.1
commit ddcab8af51c8b0a787dc630bce3a0b34c446c973

d26ed48..ffc784c 389-ds-base-1.2.11 -> 389-ds-base-1.2.11
commit ffc784c

Metadata Update from @mreynolds:
- Issue assigned to mreynolds
- Issue set to the milestone: 1.2.11.33

7 years ago

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/389ds/389-ds-base/issues/1294

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @spichugi:
- Issue close_status updated to: wontfix (was: Fixed)

3 years ago

Login to comment on this ticket.

Metadata