stack table's uuid primary key wastes resources in other tables

Bug #1263787 reported by Clint Byrum
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Heat
Triaged
Medium
Sirushti Murugesan

Bug Description

The stack table should just have an integer primary key. Every event, for instance, has to have 36 bytes of UUID instead of a 4 byte integer pointer back to the stack. The same is true of every resource record and others as well.

The UUID should just be a secondary unique index.

Revision history for this message
Steve Baker (steve-stevebaker) wrote :

Setting this to High priority

Changed in heat:
importance: Medium → High
milestone: none → icehouse-2
Changed in heat:
milestone: icehouse-2 → icehouse-3
Changed in heat:
milestone: icehouse-3 → none
Changed in heat:
assignee: nobody → Visnusaran Murugan (visnusaran-murugan)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

Fix proposed to branch: master
Review: https://review.openstack.org/106054

Changed in heat:
status: Triaged → In Progress
Changed in heat:
assignee: Visnusaran Murugan (visnusaran-murugan) → Unmesh Gurjar (unmesh-gurjar)
Changed in heat:
assignee: Unmesh Gurjar (unmesh-gurjar) → Visnusaran Murugan (visnusaran-murugan)
Changed in heat:
assignee: Visnusaran Murugan (visnusaran-murugan) → Unmesh Gurjar (unmesh-gurjar)
Changed in heat:
assignee: Unmesh Gurjar (unmesh-gurjar) → Visnusaran Murugan (visnusaran-murugan)
Zane Bitter (zaneb)
Changed in heat:
milestone: none → next
importance: High → Medium
Revision history for this message
Sirushti Murugesan (sirushtim) wrote :

So it seems like this is a necessity now for the current convergence architecture because of how the SyncPoint database schema is modeled. Reassigning this to myself because no activity for ~1 year.

Changed in heat:
assignee: Visnusaran Murugan (visnusaran-murugan) → Sirushti Murugesan (sirushtim)
Revision history for this message
Steve Baker (steve-stevebaker) wrote :

Setting this back to High, since it is needed for convergence

Changed in heat:
importance: Medium → High
Changed in heat:
status: In Progress → Triaged
Revision history for this message
Zane Bitter (zaneb) wrote :

I guess this wasn't really _needed_ for convergence, because we now have convergence but we don't have this :/

Setting it back to Medium.

Changed in heat:
importance: High → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on heat (master)

Change abandoned by Steve Baker (<email address hidden>) on branch: master
Review: https://review.openstack.org/106054
Reason: This is a very old review which has unaddressed review feedback, failing tests, or merge conflicts.

For this reason it is being abandoned to make the review queue more maintainable. It can be restored and refreshed at any time.

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

Other bug subscribers

Remote bug watches

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