ctdbd: update comment describing ctdb_call_send_redirect()
authorMichael Adam <obnox@samba.org>
Fri, 17 May 2013 09:00:32 +0000 (11:00 +0200)
committerMichael Adam <obnox@samba.org>
Mon, 26 Aug 2013 11:35:49 +0000 (13:35 +0200)
Signed-off-by: Michael Adam <obnox@samba.org>
(cherry picked from commit 9a21d417c51fb9cad8f2e87e00ca54d379aef860)

Conflicts:

server/ctdb_call.c

server/ctdb_call.c

index 48127767b2475caa3f3309fd9d198e605865f684..3ba2e8d2590713c688ded3c0c20c16fa5822fddd 100644 (file)
@@ -108,20 +108,9 @@ static void ctdb_send_error(struct ctdb_context *ctdb,
  * to its local ctdb (ctdb_request_call). If the node is not itself
  * the record's DMASTER, it first redirects the packet to  the
  * record's LMASTER. The LMASTER then redirects the call packet to
- * the current DMASTER. But there is a race: The record may have
- * been migrated off the DMASTER while the redirected packet is
- * on the wire (or in the local queue). So in case the record has
- * migrated off the new destinaton of the call packet, instead of
- * going back to the LMASTER to get the new DMASTER, we try to
- * reduce rountrips by fist chasing the record a couple of times
- * before giving up the direct chase and finally going back to the
- * LMASTER (again). Note that this works because of this: When
+ * the current DMASTER. Note that this works because of this: When
  * a record is migrated off a node, then the new DMASTER is stored
  * in the record's copy on the former DMASTER.
- *
- * The maxiumum number of attempts for direct chase to make before
- * going back to the LMASTER is configurable by the tunable
- * "MaxRedirectCount".
  */
 static void ctdb_call_send_redirect(struct ctdb_context *ctdb, 
                                    TDB_DATA key,