diff options
| author | Aymerich, Edward <edward.aymerich@hpe.com> | 2015-11-27 16:57:07 +0000 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2015-11-30 10:53:15 -0800 |
| commit | cd26c1dc415fe492fb5d14443a249f5485ad24bf (patch) | |
| tree | d3a795d4870ae8b86e3663a58688cf09b8283e60 /lib/ovsdb-idl-provider.h | |
| parent | 0b41113701e9d2c80e8621f3b98641402ba39b2b (diff) | |
| download | openvswitch-cd26c1dc415fe492fb5d14443a249f5485ad24bf.tar.gz | |
ovsdb-idl: Improve ovsdb_idl_add_table() comment.
The new comment reflects with more clarity what ovsdb_idl_add_table() does.
Previous comment could be misunderstood, leading to believe that this function
replicates all columns on IDL. Hopefully this fix clarifies that columns are
not replicated, just minimal data for reference integrity is replicated.
A comment in ovsdb_idl_table_class is also modified to better reflect this
behaviour.
Signed-off-by: Edward Aymerich <edward.aymerich@hpe.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'lib/ovsdb-idl-provider.h')
| -rw-r--r-- | lib/ovsdb-idl-provider.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/ovsdb-idl-provider.h b/lib/ovsdb-idl-provider.h index 3dddf6956..099535e8a 100644 --- a/lib/ovsdb-idl-provider.h +++ b/lib/ovsdb-idl-provider.h @@ -61,7 +61,8 @@ struct ovsdb_idl_table_class { struct ovsdb_idl_table { const struct ovsdb_idl_table_class *class; unsigned char *modes; /* OVSDB_IDL_* bitmasks, indexed by column. */ - bool need_table; /* Monitor table even if no columns? */ + bool need_table; /* Monitor table even if no columns are selected + * for replication. */ struct shash columns; /* Contains "const struct ovsdb_idl_column *"s. */ struct hmap rows; /* Contains "struct ovsdb_idl_row"s. */ struct ovsdb_idl *idl; /* Containing idl. */ |