Re: [ovs-dev] [PATCH] vtep: Make stats and status columns ephemeral.

2015-10-14 Thread Justin Pettit
> On Oct 14, 2015, at 1:06 AM, Bruce Davie wrote: > > I'm happy with this. Thank you. I'll push this with Ben's ack in just a minute. --Justin ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] vtep: Make stats and status columns ephemeral.

2015-10-14 Thread Bruce Davie
I'm happy with this. Bruce > On Oct 14, 2015, at 1:59 AM, Justin Pettit wrote: > > These fields don't need to be stored, and it causes a lot of unnecessary > writes to the database log. > > This commit also fixes a couple of trivial indentation issues with > previous ephemeral declarations. >

Re: [ovs-dev] [PATCH] vtep: Make stats and status columns ephemeral.

2015-10-13 Thread Justin Pettit
> On Oct 13, 2015, at 5:24 PM, Ben Pfaff wrote: > > On Tue, Oct 13, 2015 at 04:58:42PM -0700, Justin Pettit wrote: >> These fields don't need to be stored, and it causes a lot of unnecessary >> writes to the database log. >> >> This commit also fixes a couple of trivial indentation issues with

Re: [ovs-dev] [PATCH] vtep: Make stats and status columns ephemeral.

2015-10-13 Thread Justin Pettit
> On Oct 13, 2015, at 5:25 PM, Ben Pfaff wrote: > > On Tue, Oct 13, 2015 at 04:58:42PM -0700, Justin Pettit wrote: >> Signed-off-by: Justin Pettit > > Your Signed-off-by is wrong. Yeah, I noticed that after I sent it. I fixed it. --Justin ___ de

Re: [ovs-dev] [PATCH] vtep: Make stats and status columns ephemeral.

2015-10-13 Thread Ben Pfaff
On Tue, Oct 13, 2015 at 04:58:42PM -0700, Justin Pettit wrote: > Signed-off-by: Justin Pettit Your Signed-off-by is wrong. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] vtep: Make stats and status columns ephemeral.

2015-10-13 Thread Ben Pfaff
On Tue, Oct 13, 2015 at 04:58:42PM -0700, Justin Pettit wrote: > These fields don't need to be stored, and it causes a lot of unnecessary > writes to the database log. > > This commit also fixes a couple of trivial indentation issues with > previous ephemeral declarations. > > Signed-off-by: Just