systemtap and callgrind analysis show a lot of time spent in vattr_map_namespace_sp_getlist and slapi_vattr_namespace_values_get_sp - every time a search entry is compared against a search filter, and then again when the entry is sent to the client, to see if there are any virtual attributes to add - if a user does not want or care about virtual attributes at all, there should be a way to turn off this vattr checking using a config attribute
attachment 0001-Ticket-511-allow-turning-off-vattr-lookup-in-search-.patch
git merge ticket511 Updating 6ee822f..1e2745f Fast-forward ldap/servers/slapd/libglobs.c | 27 +++++++++++++++++++++++++++ ldap/servers/slapd/proto-slap.h | 2 ++ ldap/servers/slapd/slap.h | 4 ++++ ldap/servers/slapd/vattr.c | 28 +++++++++++++++++++--------- 4 files changed, 52 insertions(+), 9 deletions(-)
[mareynol@localhost replication]$ git push origin master Counting objects: 17, done. Delta compression using up to 4 threads. Compressing objects: 100% (9/9), done. Writing objects: 100% (9/9), 1.60 KiB, done. Total 9 (delta 7), reused 0 (delta 0) To ssh://git.fedorahosted.org/git/389/ds.git 6ee822f..1e2745f master -> master
Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=918705
Remove Slapi_Counter to store the ncofnig setting 0001-Ticket-511-Revision-allow-turning-off-vattr-lookup-i.patch
git merge ticket511 Updating ba0d579..b3d2ee0 Fast-forward ldap/servers/slapd/libglobs.c | 13 +++++-------- ldap/servers/slapd/slap.h | 4 +--- 2 files changed, 6 insertions(+), 11 deletions(-)
git push origin master Counting objects: 13, done. Delta compression using up to 4 threads. Compressing objects: 100% (7/7), done. Writing objects: 100% (7/7), 914 bytes, done. Total 7 (delta 5), reused 0 (delta 0) To ssh://git.fedorahosted.org/git/389/ds.git ba0d579..b3d2ee0 master -> master
commit b3d2ee0
Metadata Update from @mreynolds: - Issue assigned to mreynolds - Issue set to the milestone: 1.3.1
Just for recording, I think this ticket was impacted by glibc issue https://bugzilla.redhat.com/show_bug.cgi?id=200247
Metadata Update from @tbordaz: - Issue close_status updated to: None (was: Fixed) - Issue set to the milestone: None (was: 1.3.1)
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/511
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
Login to comment on this ticket.