Juniper Openstack

Change data-structure used for waitq_ in task library

  1. Series r2.20
  2. Bug #1462318
Bug #1462318 reported by Praveen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
Medium
Praveen
Juniper Openstack r2.20-fcs
Trunk
Fix Committed
Medium
Praveen
Juniper Openstack r3.0-fcs

Bug Description

In scaled scenarios, its is observed that length of waitq in control-node can be in thousands. The current task library uses vector to store all Task's in waitq. When an entry is removed from waitq, the erase operation on a vector can have significant overhead.

Must modify the waitq_ to use tree instead of vectors.

Tags: base
Praveen (praveen-karadakal)
tags: added: base
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/11317
Submitter: Praveen K V (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/11317
Committed: http://github.org/Juniper/contrail-controller/commit/14e4b8e7080d09954c72342f3b828952c949f0b4
Submitter: Zuul
Branch: R2.20

commit 14e4b8e7080d09954c72342f3b828952c949f0b4
Author: Praveen K V <email address hidden>
Date: Fri Jun 5 21:58:46 2015 +0530

Task library optimizations

Modified task library to use boost::intrusive::list instead of vector

Change-Id: I0626387915d159bd5cdaf53d5f352374265f5037
Closes-Bug: #1462318

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/11345
Submitter: Praveen K V (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/11345
Committed: http://github.org/Juniper/contrail-controller/commit/a43a0edbb02ce830cf3e35ee3e21bf0b45c04cb6
Submitter: Zuul
Branch: master

commit a43a0edbb02ce830cf3e35ee3e21bf0b45c04cb6
Author: Praveen K V <email address hidden>
Date: Fri Jun 5 21:58:46 2015 +0530

Task library optimizations

Modified task library to use boost::intrusive::list instead of vector

Change-Id: I0626387915d159bd5cdaf53d5f352374265f5037
Closes-Bug: #1462318

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.