[Engine-devel] Engine table comments

Omer Frenkel ofrenkel at redhat.com
Wed Feb 27 09:26:40 UTC 2013


----- Original Message -----

> From: "Libor Spevak" <lspevak at redhat.com>
> To: "Yair Zaslavsky" <yzaslavs at redhat.com>
> Cc: engine-devel at ovirt.org
> Sent: Wednesday, February 27, 2013 9:36:16 AM
> Subject: Re: [Engine-devel] Engine table comments

> Thanks for the confirmation. Appending a template for the columns :-)

> psql engine -U postgres -c "SELECT 'COMMENT ON COLUMN ' || table_name
> || '.' || column_name || ' IS TODO;' sql FROM
> information_schema.columns where table_schema='public' order by
> table_name, column_name" > comments_columns_to_do.sql

> It looks like we have cca 73 tables and 2353 columns now. So probably
> better to start with the tables.

i documented some (not complete) information about columns of vm_static here: 
http://wiki.ovirt.org/Features/Templates-3.2#Design 
as part of preparation for instance types feature 
so maybe we can use it as well 

> On 26.2.2013 15:03, Yair Zaslavsky wrote:

> > I think your idea is very good.
> 
> > I also think we should have it for columns as well.
> 
> > But since we already have many columns, maybe we should have an "on
> > going effort" from various developers on this (i.e - each developer
> > will do that for several tables).
> 
> > Thoughts?
> 

> > ----- Original Message -----
> 

> > > From: "Libor Spevak" <lspevak at redhat.com>
> > 
> 
> > > To: engine-devel at ovirt.org
> > 
> 
> > > Sent: Tuesday, February 26, 2013 2:37:17 PM
> > 
> 
> > > Subject: [Engine-devel] Engine table comments
> > 
> 

> > > Hi,
> > 
> 
> > > I would like to propose to create a patch assigning sql comment
> > > to
> > > each Engine table.
> > 
> 
> > > The comments will be a part of the exported DB schema report,
> > > too.
> > 
> 
> > > The terminology is not always consistent across all application
> > > layers, so for the newcomers it would be easier to understand the
> > > model.
> > 
> 

> > > But, what would be general guidelines?
> > 
> 

> > > I recommend to use singular form of entity name as used from
> > > end-user
> > > point of view (if somehow visible, of course), or just used
> > > frequently inside code.
> > 
> 

> > > Examples:
> > 
> 
> > > storage_pool ... Data center
> > 
> 
> > > vds_groups ... Cluster
> > 
> 
> > > but:
> > 
> 
> > > vm_static ... Virtual machine configuration
> > 
> 
> > > vm_dynamic ... Virtual machine runtime data
> > 
> 
> > > vm_statistics ... Virtual machine statistics data
> > 
> 
> > > ...
> > 
> 
> > > etc.
> > 
> 

> > > This command prepares a template for the patch:
> > 
> 

> > > psql engine -U postgres -c "select 'COMMENT ON TABLE ' || relname
> > > ||
> > > ' IS ''TODO'';' sql from pg_stat_user_tables t WHERE
> > > schemaname='public' order by t.relname" > comments_to_do.sql
> > 
> 

> > > or just I can prepare a page on the wiki with table containing
> > > table
> > > names and volunteers :-) can propose simple understandable
> > > comments
> > > for known tables.
> > 
> 

> > > Is it useful?
> > 
> 

> > > Thanks.
> > 
> 

> > > Regards,
> > 
> 
> > > Libor
> > 
> 

> > > _______________________________________________
> > 
> 
> > > Engine-devel mailing list
> > 
> 
> > > Engine-devel at ovirt.org
> > 
> 
> > > http://lists.ovirt.org/mailman/listinfo/engine-devel
> > 
> 

> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/engine-devel/attachments/20130227/70acb4f0/attachment.html>


More information about the Engine-devel mailing list