Do not retry on 413 response codes

Previously when Elasticsearch responds with a 413 (Payload Too Large) status,
the manticore adapter raises an error before the response can be processed
by the bulk_send error handling. This commit updates the adpapter to pass
through the response code without raising an exception so that it can be
properly handled. NOTE: this only applies to code 413 even though there may be
arguably other error codes that should not be retried. The scope of this work is
simply to 413 where we have explicit handling.

Author: donoghuc

Diff for: lib/logstash/outputs/elasticsearch/http_client/manticore_adapter.rb

@@ -77,10 +77,11 @@ def perform_request(url, method, path, params={}, body=nil)
       end
 
       # 404s are excluded because they are valid codes in the case of
-      # template installation. We might need a better story around this later
-      # but for our current purposes this is correct
+      # template installation. 413s are excluded to allow the bulk_send
+      # error handling to process "Payload Too Large" responses rather
+      # than triggering retries.
       code = resp.code
-      if code < 200 || code > 299 && code != 404
+      if code < 200 || (code > 299 && ![404, 413].include?(code))
         raise ::LogStash::Outputs::ElasticSearch::HttpClient::Pool::BadResponseCodeError.new(code, request_uri, body, resp.body)
       end