Skip to content

Commit

Permalink
Merge pull request grpc#7697 from ejona86/waitforready
Browse files Browse the repository at this point in the history
Rename Fail Fast doc to Wait for Ready
  • Loading branch information
ctiller authored Aug 31, 2016
2 parents 581367a + cf88bb4 commit a3328ae
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 15 deletions.
16 changes: 1 addition & 15 deletions doc/fail_fast.md
Original file line number Diff line number Diff line change
@@ -1,15 +1 @@
gRPC Fail Fast Semantics
========================

Fail fast requests allow terminating requests (with status UNAVAILABLE) prior
to the deadline of the request being met.

gRPC implementations of fail fast can terminate requests whenever a channel is
in the TRANSIENT_FAILURE or SHUTDOWN states. If the channel is in any other
state (CONNECTING, READY, or IDLE) the request should not be terminated.

Fail fast SHOULD be the default for gRPC implementations, with an option to
switch to non fail fast.

The opposite of fail fast is 'ignore connectivity'.

Moved to wait-for-ready.md
14 changes: 14 additions & 0 deletions doc/wait-for-ready.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
gRPC Wait for Ready Semantics
=============================

If an RPC is issued but the channel is in `TRANSIENT_FAILURE` or `SHUTDOWN`
states, the RPC is unable to be transmited promptly. By default, gRPC
implementations SHOULD fail such RPCs immediately. This is known as "fail fast,"
but usage of the term is historical. RPCs SHOULD NOT fail as a result of the
channel being in other states (`CONNECTING`, `READY`, or `IDLE`).

gRPC implementations MAY provide a per-RPC option to not fail RPCs as a result
of the channel being in `TRANSIENT_FAILURE` state. Instead, the implementation
queues the RPCs until the channel is `READY`. This is known as "wait for ready."
The RPCs SHOULD still fail before `READY` if there are unrelated reasons, such
as the channel is `SHUTDOWN` or the RPC's deadline is reached.

0 comments on commit a3328ae

Please sign in to comment.