diff options
author | Han Zhou <hzhou8@ebay.com> | 2019-03-01 10:56:37 -0800 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2019-03-07 14:23:51 -0800 |
commit | 2cd62f75c1b4af1fc0786afbae6b9fa910f37f0c (patch) | |
tree | d41fb3d75918b178bb5edbe3d25b0985650efd06 /ovsdb/transaction.h | |
parent | 2a7be04ea9275317f295f66a9335f37af7e864e0 (diff) | |
download | openvswitch-2cd62f75c1b4af1fc0786afbae6b9fa910f37f0c.tar.gz |
ovsdb raft: Precheck prereq before proposing commit.
In current OVSDB Raft design, when there are multiple transactions
pending, either from same server node or different nodes in the
cluster, only the first one can be successful at once, and following
ones will fail at the prerequisite check on leader node, because
the first one will update the expected prerequisite eid on leader
node, and the prerequisite used for proposing a commit has to be
committed eid, so it is not possible for a node to use the latest
prerequisite expected by the leader to propose a commit until the
lastest transaction is committed by the leader and updated the
committed_index on the node.
Current implementation proposes the commit as soon as the transaction
is requested by the client, which results in continously retry which
causes high CPU load and waste.
Particularly, even if all clients are using leader_only to connect to
only the leader, the prereq check failure still happens a lot when
a batch of transactions are pending on the leader node - the leader
node proposes a batch of commits using the same committed eid as
prerequisite and it updates the expected prereq as soon as the first
one is in progress, but it needs time to append to followers and wait
until majority replies to update the committed_index, which results in
continously useless retries of the following transactions proposed by
the leader itself.
This patch doesn't change the design but simplely pre-checks if current
eid is same as prereq, before proposing the commit, to avoid waste of
CPU cycles, for both leader and followers. When clients use leader_only
mode, this patch completely eliminates the prereq check failures.
In scale test of OVN with 1k HVs and creating and binding 10k lports,
the patch resulted in 90% CPU cost reduction on leader and >80% CPU cost
reduction on followers. (The test was with leader election base time
set to 10000ms, because otherwise the test couldn't complete because
of the frequent leader re-election.)
This is just one of the related performance problems of the prereq
checking mechanism dicussed at:
https://mail.openvswitch.org/pipermail/ovs-discuss/2019-February/048243.html
Signed-off-by: Han Zhou <hzhou8@ebay.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'ovsdb/transaction.h')
-rw-r--r-- | ovsdb/transaction.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/ovsdb/transaction.h b/ovsdb/transaction.h index c81937357..c21871a45 100644 --- a/ovsdb/transaction.h +++ b/ovsdb/transaction.h @@ -29,6 +29,7 @@ void ovsdb_txn_set_txnid(const struct uuid *, struct ovsdb_txn *); const struct uuid *ovsdb_txn_get_txnid(const struct ovsdb_txn *); void ovsdb_txn_abort(struct ovsdb_txn *); +bool ovsdb_txn_precheck_prereq(const struct ovsdb *db); struct ovsdb_error *ovsdb_txn_replay_commit(struct ovsdb_txn *) OVS_WARN_UNUSED_RESULT; struct ovsdb_txn_progress *ovsdb_txn_propose_commit(struct ovsdb_txn *, |