> On Aug 2, 2016, at 12:09 AM, Russell Bryant <[email protected]> wrote: > > > > On Mon, Aug 1, 2016 at 12:00 PM, Ben Pfaff <[email protected] > <mailto:[email protected]>> wrote: > On Mon, Aug 01, 2016 at 02:55:26AM -0700, nickcooper-zhangtonghao wrote: > > If the chassis doesn't configure the 'external-ids:ovn-bridge-mappings' in > > the OVSDB, the 'add_bridge_mappings' should return directly. > > > > Signed-off-by: nickcooper-zhangtonghao > > <[email protected]> > > What's the benefit? > > I was looking at this earlier before I got on some calls. > > There doesn't appear to be any bug here, but this patch does skip some > unnecessary code. The most notable improvement seems to be skipping a loop > over all Port_Binding records in every call to patch_run. > > -- > Russell Bryant
I’am with Russell. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
