[ovs-dev] [PATCH 2/2] Implement database schema versioning.

Ben Pfaff blp at nicira.com
Mon Dec 27 18:11:26 UTC 2010


On Mon, Dec 27, 2010 at 09:52:48AM -0800, Peter Balland wrote:
> > +    The "version" reports the version of the database schema.  This is
> > +    a recent addition to the schema format, so OVSDB permits it to be
> > +    absent.  This is deprecated and may be removed in the future.
> > +    Open vSwitch semantics for "version" are described in
> > +    ovs-vswitchd.conf.db(5).
> 
> This makes it sound like version has been deprecated, but I don't
> think that's what you intended.

You're right, that's badly worded.

I changed the middle sentences to:

    Because this is a recent addition to the schema format, OVSDB
    permits it to be omitted, but future versions of OVSDB will require
    it to be present.

> I skimmed through the rest of the code, which seems reasonable, from
> my limited familiarity.

Thank you for taking a look.




More information about the dev mailing list