Re: [ovs-dev] [PATCH] ovsdb-server: Drop unnecessary find_db() function.

2017-03-29 Thread Ben Pfaff
On Mon, Mar 20, 2017 at 12:44:47PM -0700, Andy Zhou wrote: > On Mon, Mar 20, 2017 at 10:50 AM, Ben Pfaff wrote: > > 'all_dbs' maps from a schema name to its struct db, so there's no need to > > iterate the whole thing to find a database by schema name; instead, just > > use the shash in the usual

Re: [ovs-dev] [PATCH] ovsdb-server: Drop unnecessary find_db() function.

2017-03-20 Thread Andy Zhou
On Mon, Mar 20, 2017 at 10:50 AM, Ben Pfaff wrote: > 'all_dbs' maps from a schema name to its struct db, so there's no need to > iterate the whole thing to find a database by schema name; instead, just > use the shash in the usual way. > > Also, avoid trying too hard in ovsdb_replication_init() in

[ovs-dev] [PATCH] ovsdb-server: Drop unnecessary find_db() function.

2017-03-20 Thread Ben Pfaff
'all_dbs' maps from a schema name to its struct db, so there's no need to iterate the whole thing to find a database by schema name; instead, just use the shash in the usual way. Also, avoid trying too hard in ovsdb_replication_init() in a similar way. Signed-off-by: Ben Pfaff --- ovsdb/ovsdb-s