On Thu, Jun 28, 2012 at 04:40:37PM -0400, Justin Pettit wrote:
> On Jun 28, 2012, at 1:35 PM, Ben Pfaff wrote:
> 
> > I don't understand why the priority field sort order is the opposite
> > of other fields.  I'd drop that.
> 
> I had suggested this after seeing some of his initial output.  I
> imagine most people will just run "--sort" and expect to see flows
> sorted from highest to lowest priority.  I realize this introduces
> an inconsistency, so I was sort of on the fence about it.  It sounds
> like you have a stronger preference than I, so I'll defer to you.

The documentation for the current way just jumps out as weird to me.
It's basically "--sort sorts in ascending order unless it's the
priority field in which case it sorts in descending order".  I can see
that priority might be a little different in expectations, but I think
that if I used --sort and then decided that I wanted it sorted the
other way I'd just insert an 'r'.

> (I still think the secondary sort should always sort highest to
> lowest by priority, though.  Thoughts?)

I'm OK with that.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to