[ovs-dev] [PATCH 4/4] ovn-nbctl: Enable database commands using db-ctl-base infrastructure.
Ben Pfaff
blp at nicira.com
Tue Sep 15 21:44:47 UTC 2015
On Tue, Sep 15, 2015 at 10:22:01AM -0700, Justin Pettit wrote:
>
> > On Sep 14, 2015, at 1:27 PM, Russell Bryant <rbryant at redhat.com> wrote:
> >
> > On 09/14/2015 04:14 PM, ALeX Wang wrote:
> >> On 14 September 2015 at 08:33, Ben Pfaff <blp at nicira.com> wrote:
> >>
> >>> I'm not sure what you mean. The cache that ovs-vsctl implements is only
> >>> a cache; that is, leaving it out it doesn't affect correctness. On the
> >>> other hand, if there is a cache, then logic is needed to invalidate the
> >>> cache if anything changes.
> >>>
> >>> Maybe you could give an example of the problem you see?
> >>>
> >>>
> >>
> >> Sorry, I was confused to believe that cache is compulsory,... it is
> >> just used for easy access, using hash.
> >>
> >> Then, looks good to me~ maybe Russell would also like to look at this?
> >
> > Thanks for asking, but no need. I think the idea of converting to
> > db-ctl-base makes perfect sense. I'm happy if the code looks good to you.
> >
> > I use it a good bit when testing things, so I'll be sure to report or
> > fix anything i run in to. :-)
>
> Ben, it doesn't look like you've checked this in. Are you still
> waiting on an ACK? Alex, did you do a full review?
I was just too busy with unrelated nonsense. It's in now.
More information about the dev
mailing list