ovsdb: Fix one off error in tracking monitor changes
authorAndy Zhou <azhou@ovn.org>
Mon, 22 Feb 2016 08:24:06 +0000 (00:24 -0800)
committerAndy Zhou <azhou@ovn.org>
Wed, 24 Feb 2016 23:54:50 +0000 (15:54 -0800)
commit5e22917729791c66ea9f5dd74b36858fa73b1b28
treeed4959f0a88e115e646c6d2d119320e33088a9ea
parent655306c825a8296573f9ee9f628affe9c4345c5f
ovsdb: Fix one off error in tracking monitor changes

dbmon's changes should be stored with the next transaction number,
rather than the current transaction number.  This bug causes the
changes of a transaction stored in a monitor to be unnoticed by
the jsonrpc connections that is responsible for flush the monitor
content.

However, the bug was not noticed until it was exposed by a later
optimization patch: "avoid unnecessary call to ovsdb_monitor_get_update()."
The lack of optimization means that the update is still generated
when 'unflushed' equals to n_transactions + 1, which should have
indicated the monitor has been flushed already.

Signed-off-by: Andy Zhou <azhou@ovn.org>
Acked-by: Liran Schour <lirans@il.ibm.com>
Acked-by: Ben Pfaff <blp@ovn.org>
ovsdb/monitor.c