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); }}}
revision 0001-Ticket-47963-RFE-memberOf-add-option-to-skip-nested.patch
Replying to [comment:2 rmeggins]:
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
Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1167976
attachment 0001-Ticket-47963-skip-nested-groups-breaks-memberof-fixu.patch
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
Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1174457
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.
Fix regression from previous patches 0001-Ticket-47963-memberof-skip-nested-groups-breaks-the-.patch
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
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.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: Fixed)
Login to comment on this ticket.