There should be a naming convention for neutron DB tables

Bug #1347361 reported by Henry Gessau
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Won't Fix
Low
Henry Gessau

Bug Description

Now that the database is healed and all tables are present, the names of tables are haphazard and there is no convention for avoiding naming conflicts or having sensible grouping.

A naming convention may look something like:

<vendor>_<plugin|service>_<feature|purpose>

Existing tables should be renamed to follow the convention.

A README file explaining the convention should be created in the directory with the models.

Tags: db
Henry Gessau (gessau)
Changed in neutron:
assignee: nobody → Henry Gessau (gessau)
Revision history for this message
Kevin Benton (kevinbenton) wrote :

Why should existing tables be renamed? Won't that create quite a bit of unnecessary work in terms of migrations and code review?

Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

I agree with Kevin here.
That mostly should be applied to vendor-specific models

tags: added: db
Changed in neutron:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Ann Taraday (akamyshnikova) wrote :

In fact some of tables already have name of plugin in its name. Naming new tables according to this naming conversion is a good idea, will keep this in mind while doing reviews.

goocher (farmerworking)
Changed in neutron:
status: Confirmed → In Progress
Changed in neutron:
status: In Progress → Confirmed
Revision history for this message
Henry Gessau (gessau) wrote :

The recommended naming of tables for sub-projects is documented. The risk of conflicting names is low enough that it is not worth the effort to implement checks across sub-projects. Marking as Won't-Fix. This can be revisited if it becomes a problem.

Changed in neutron:
status: Confirmed → Won't Fix
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.