ctdbd: update comment describing ctdb_call_send_redirect()
authorMichael Adam <obnox@samba.org>
Fri, 17 May 2013 09:00:32 +0000 (11:00 +0200)
committerAmitay Isaacs <amitay@gmail.com>
Fri, 24 May 2013 12:06:16 +0000 (22:06 +1000)
Signed-off-by: Michael Adam <obnox@samba.org>
server/ctdb_call.c

index dbbebec8c8f51bd800b56347de29415df419f06a..4696d27e022fdf97707788e28b64fa397279d959 100644 (file)
@@ -113,20 +113,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 round-trips by first 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 maximum 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,
                                    struct ctdb_db_context *ctdb_db,