Bug 1334926 - [RFE] Execute command callbacks in parallel (to improve performance)
Summary: [RFE] Execute command callbacks in parallel (to improve performance)
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Infra
Version: 4.0.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Moti Asayag
QA Contact: Lukas Svaty
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-05-10 19:59 UTC by Moti Asayag
Modified: 2022-02-23 14:26 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-03-12 16:24:02 UTC
oVirt Team: Infra
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHV-44795 0 None None None 2022-02-23 14:26:52 UTC

Description Moti Asayag 2016-05-10 19:59:01 UTC
Description of problem:
The Command Coordinator (CoCo) added the ability to process callbacks after the execution of the command. The callback may be invoked either by periodic polling or by subscribing to an event.

However, processing of the callbacks is done serially, causing some delay in the execution of the callbacks. The mechanism which processes the callbacks can be improved to execute the callbacks in parallel by introducing a thread pool to submit the callbacks for the execution.

Comment 1 Yaniv Kaul 2017-06-06 19:17:19 UTC
Oved - do we feel we have the resource to implement it? Otherwise, please either defer or close-deferred.

Comment 2 Oved Ourfali 2017-06-07 05:57:06 UTC
I think that's important gap to implement.
Martin - your thoughts?

Comment 3 Martin Perina 2017-06-07 08:39:03 UTC
It may improve performance, so let's decide about possible target once we know the 4.2 release schedule and available resources for it.

Comment 4 Ravi Nori 2018-01-02 15:14:00 UTC
Invoking callbacks in parallel may not be as simple as it sounds. Additional code changes and testing will be required to make sure we don't have any regression. The additional code changes would be to make sure that the command's end method is not invoked in another iteration of polling before the first one can complete.

Comment 5 Martin Perina 2019-03-12 16:24:02 UTC
Closing due to limited resources


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