API v1 listing of records noticibly slower moving to Juno
Bug #1413472 reported by
Sam Morrison
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Designate |
Fix Released
|
High
|
Graham Hayes | ||
Bug Description
Doing:
time designate record-list XXXX-XXXX-
With a domain with 100 records:
Icehouse:
real 0m0.842s
user 0m0.320s
sys 0m0.046s
Juno:
real 0m1.574s
user 0m0.348s
sys 0m0.031s
With a domain with 1000 records:
Icehouse:
real 0m3.263s
user 0m1.576s
sys 0m0.067s
Juno:
real 0m9.180s
user 0m1.574s
sys 0m0.042s
I can see this is due to the code in storage/
first it gets all record sets then does a for loop over them and does another sql query. I wonder if this query could be made more efficient by using one query and a join?
| Changed in designate: | |
| status: | New → Confirmed |
| importance: | Undecided → Medium |
| milestone: | none → kilo-2 |
| Changed in designate: | |
| milestone: | kilo-2 → kilo-3 |
| Changed in designate: | |
| milestone: | kilo-3 → kilo-rc1 |
| Changed in designate: | |
| importance: | Medium → High |
| summary: |
- Listing records noticibly slower moving to Juno + API v1 listing of records noticibly slower moving to Juno |
| Changed in designate: | |
| assignee: | nobody → Graham Hayes (grahamhayes) |
| Changed in designate: | |
| milestone: | kilo-rc1 → liberty-1 |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to designate (master) | #1 |
| Changed in designate: | |
| status: | Confirmed → In Progress |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to designate (master) | #2 |
| Changed in designate: | |
| status: | In Progress → Fix Committed |
| Changed in designate: | |
| status: | Fix Committed → Fix Released |
| Changed in designate: | |
| milestone: | liberty-1 → 1.0.0 |
To post a comment you must log in.
Fix proposed to branch: master
Review: https:/