Re: [ovs-dev] [PATCH v2 17/21] ovn: Rename Pipeline table to Rule table.

2015-08-03 Thread Ben Pfaff
I applied this patch and all the previous ones to master, with that naming change. On Sun, Aug 02, 2015 at 12:12:48PM -0700, Ben Pfaff wrote: > OK, I'll change this to use Logical_Flow for the name of the table. > > On Fri, Jul 31, 2015 at 03:40:19PM -0700, Alex Wang wrote: > > Just to mention th

Re: [ovs-dev] [PATCH v2 17/21] ovn: Rename Pipeline table to Rule table.

2015-08-02 Thread Ben Pfaff
OK, I'll change this to use Logical_Flow for the name of the table. On Fri, Jul 31, 2015 at 03:40:19PM -0700, Alex Wang wrote: > Just to mention that I also think Flow is a better name for this table. > > On Fri, Jul 31, 2015 at 11:19 AM, Ben Pfaff wrote: > > > I have a lot of misgivings about

Re: [ovs-dev] [PATCH v2 17/21] ovn: Rename Pipeline table to Rule table.

2015-07-31 Thread Alex Wang
Just to mention that I also think Flow is a better name for this table. On Fri, Jul 31, 2015 at 11:19 AM, Ben Pfaff wrote: > I have a lot of misgivings about all of the possible names. > > Thanks. > > On Thu, Jul 30, 2015 at 04:33:23PM -0700, Justin Pettit wrote: > > I think Pipeline is more des

Re: [ovs-dev] [PATCH v2 17/21] ovn: Rename Pipeline table to Rule table.

2015-07-31 Thread Ben Pfaff
I have a lot of misgivings about all of the possible names. Thanks. On Thu, Jul 30, 2015 at 04:33:23PM -0700, Justin Pettit wrote: > I think Pipeline is more descriptive about what it actually is. I also find > it confusing since we use the term "rule" in the classifier. I think Flow > (or Lo

Re: [ovs-dev] [PATCH v2 17/21] ovn: Rename Pipeline table to Rule table.

2015-07-30 Thread Justin Pettit
I think Pipeline is more descriptive about what it actually is. I also find it confusing since we use the term "rule" in the classifier. I think Flow (or Logical_Flow) would be clearer than Rule, since we really are talking about flows, and people may look for a distinction that isn't there.