[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [OSSTEST PATCH 5/8] Schema: Support database schema updates

Ian Campbell writes ("Re: [OSSTEST PATCH 5/8] Schema: Support database schema 
> On Thu, 2015-12-10 at 18:27 +0000, Ian Jackson wrote:
> > See schema/README.schema, introduced in this patch, for the design.
> The README looks good now, thanks.


> Code comments....

> > diff --git a/mg-schema-create b/mg-schema-create
> Why remove the safety catch in this context?

The safety catch breaks when run on an empty schema, because there is
no `tasks' table.  Setting this pattern to `.*' triggers a special
case which avoids even trying to see what test DBs exist.

And furthermore the safety catch is not needed because if we are
mistakenly operating on an existing database (test or real) we bomb
out as previously discussed.

I can put this in a comment if you like.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.