Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allow turning off vattr lookup in search entry return #511

Closed
389-ds-bot opened this issue Sep 12, 2020 · 8 comments
Closed

allow turning off vattr lookup in search entry return #511

389-ds-bot opened this issue Sep 12, 2020 · 8 comments

Comments

@389-ds-bot
Copy link

Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/511


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

@389-ds-bot
Copy link
Author

@389-ds-bot
Copy link
Author

Comment from mreynolds (@mreynolds389) at 2012-12-13 03:47:25

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

@389-ds-bot
Copy link
Author

Comment from nkinder (@nkinder) at 2013-03-07 00:15:27

Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=918705

@389-ds-bot
Copy link
Author

Comment from mreynolds (@mreynolds389) at 2013-05-08 03:11:24

Remove Slapi_Counter to store the ncofnig setting
0001-Ticket-511-Revision-allow-turning-off-vattr-lookup-i.patch

@389-ds-bot
Copy link
Author

Comment from mreynolds (@mreynolds389) at 2013-05-08 20:50:50

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

@389-ds-bot
Copy link
Author

Comment from mreynolds (@mreynolds389) at 2017-02-11 23:05:36

Metadata Update from @mreynolds389:

  • Issue assigned to mreynolds389
  • Issue set to the milestone: 1.3.1

@389-ds-bot
Copy link
Author

Comment from tbordaz (@tbordaz) at 2018-07-30 10:48:17

Just for recording, I think this ticket was impacted by glibc issue https://bugzilla.redhat.com/show_bug.cgi?id=200247

@389-ds-bot
Copy link
Author

Comment from tbordaz (@tbordaz) at 2018-07-30 10:48:18

Metadata Update from @tbordaz:

  • Issue close_status updated to: None (was: Fixed)
  • Issue set to the milestone: None (was: 1.3.1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant