389-ds-base

#50024 Ticket 49574 - remove index subsystem
Closed by spichugi. Opened by lkrispen.
lkrispen/389-ds-base t49574  into  master

Download 50024.patch

There is code for an indexing subsystem, which looks like it was intended
to provide special indexes for specific search filters (not vlv).
But it is not used and not documented and unclear if it ever worked or will
work. It only contains calls in search processing to check if something is defined
and so only provides a bit of overhead in searches.

We can remove this dead code. We can also remove the broker api for idl_xxx since it is not used
anywhere else.

I'd say it makes sense to run all the index tests we have in TET with it and the full python test run. Just to check for the regressions.
Depending on it, we can set 'ack'.

@vashirov could you please check TET? I'll run the full Python both with and without the PR.

The python test suites haven't shown any regressions.

Fantastic! I'm really excited to see this clean up in progress :)

LGTM! And is this your first PR? :-)

rebased onto 8d51c979f247c90bc0d005ec4edc821558ee7a28

Pull-Request has been merged by lkrispen

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 pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3083

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.