Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1044213 - backend performance - introduce optimization levels
backend performance - introduce optimization levels
Status: CLOSED ERRATA
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: 389-ds-base (Show other bugs)
7.0
Unspecified Unspecified
low Severity unspecified
: rc
: ---
Assigned To: Rich Megginson
Viktor Ashirov
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2013-12-17 17:26 EST by Nathan Kinder
Modified: 2015-03-05 04:33 EST (History)
4 users (show)

See Also:
Fixed In Version: 389-ds-base-1.3.3.1-1.el7
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2015-03-05 04:33:12 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2015:0416 normal SHIPPED_LIVE Important: 389-ds-base security, bug fix, and enhancement update 2015-03-05 09:26:33 EST

  None (edit)
Description Nathan Kinder 2013-12-17 17:26:45 EST
This bug is created as a clone of upstream ticket:
https://fedorahosted.org/389/ticket/47358

modify performance is highly impacted by the backend lock and entry cache lock contention.
A couple of tickets are related to this and suggest changes to improve performance, but 
- it is not guranteed that each optimization id good for all deployments
- it is a sensitive area and there is a possibility of unexpected side effects 

To move forward I suggest to implement some backend related optimizations, but allow to turn them on or off, so it is possible to keep the current behaviour and deploy the optimizations after testing in a specific deployment.

For the beginning, I think the following optimizations should be considered:

- do not update the database ruv in the modify transaction, let it be done by the calls to _replica_update_state in the event loop. This is handled specifically by#564, but it is not finally commited and some environments may want to keep the ruv update in the modify txn

- reverse order of txn_begin/commit and backend_lock/unlock:
with the fix for #568 it is save to use batching of transaction log flush, but if the transaction is nested inside the backend lock, always only one transaction will be active, so the effect is not visible

- move code out of the backend lock:
in a modify operation there is a lot of initial work done after taking the backend lock, but not everything requires this protection as no database access is involved, eg
-- check for illegal mods
-- check access to the entry
-- make copies of the mods to restore in a retry
-- make copies of the entry for use in plugins
some of the above depend on the enty already be found and locked in the entry cache (the entrycache itself is addressed in performance tickets #414,#614).
There is no obvious reason that the find_entry2modify cannot be done before taking the backend lock, but this would request thorough investigation and testing. For testing it could help to have a switch to turn this behaviour on
Comment 8 Noriko Hosoi 2015-01-30 12:53:45 EST
Hi Viktor, Hi Ami,

I agree with Viktor that there are some issues in the test cases... (I'm referring CLU now.)

It looks to me there's no impact from the optimization level, though.  So, I believe we can set VERIFIED to this bug.  Looking into the tet test results...

Thanks!!
--noriko
Comment 9 Viktor Ashirov 2015-01-30 13:19:53 EST
Thank you so much, Noriko! 

Marking as VERIFIED.
Comment 11 errata-xmlrpc 2015-03-05 04:33:12 EST
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHSA-2015-0416.html

Note You need to log in before you can comment on or make changes to this bug.