Permalink
Browse files

stop retrying on ProtocolException

  • Loading branch information...
1 parent 49f1269 commit aa522a6bdc1cf6b9d34355e3de5015b0ad6e39a4 @ryanking ryanking committed Sep 17, 2010
Showing with 1 addition and 2 deletions.
  1. +1 −1 lib/thrift_client.rb
  2. +0 −1 lib/thrift_client/abstract_thrift_client.rb
View
@@ -24,7 +24,7 @@ class NoServersAvailable < StandardError; end
<tt>:protocol_extra_params</tt>:: An array of additional parameters to pass to the protocol initialization call. Defaults to <tt>[]</tt>.
<tt>:transport</tt>:: Which Thrift transport to use. Defaults to <tt>Thrift::Socket</tt>.
<tt>:transport_wrapper</tt>:: Which Thrift transport wrapper to use. Defaults to <tt>Thrift::FramedTransport</tt>.
-<tt>:exception_classes</tt>:: Which exceptions to catch and retry when sending a request. Defaults to <tt>[IOError, Thrift::Exception, Thrift::ProtocolException, Thrift::ApplicationException, Thrift::TransportException, NoServersAvailable]</tt>
+<tt>:exception_classes</tt>:: Which exceptions to catch and retry when sending a request. Defaults to <tt>[IOError, Thrift::Exception, Thrift::ApplicationException, Thrift::TransportException, NoServersAvailable]</tt>
<tt>:raise</tt>:: Whether to reraise errors if no responsive servers are found. Defaults to <tt>true</tt>.
<tt>:retries</tt>:: How many times to retry a request. Defaults to 0.
<tt>:server_retry_period</tt>:: How many seconds to wait before trying to reconnect after marking all servers as down. Defaults to <tt>1</tt>. Set to <tt>nil</tt> to retry endlessly.
@@ -16,7 +16,6 @@ def mark_down!
DISCONNECT_ERRORS = [
IOError,
Thrift::Exception,
- Thrift::ProtocolException,
Thrift::ApplicationException,
Thrift::TransportException
]

0 comments on commit aa522a6

Please sign in to comment.