noiseless 0.4.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1a2c76e38f11ce0e3d82ecea5c3644022e1b793613e413f81987063e333da393
-  data.tar.gz: 7b862a6892326ed2fff34d3aa087d2cdfeafbc1b007b5f5c6285286d12d26a4f
+  metadata.gz: 466267a77224bbf9915eff8b198002c658e2e9c9d368c6c7f6e4e4ee32939892
+  data.tar.gz: 71281cfbbcdd40f83f55eedb0fca494f1e0a75172c4a9523c3000cae81fa752d
 SHA512:
-  metadata.gz: d25d6a3a8fd986c7f99454f358da6abe29a7c64ee4beddab5925a83c580469625dfc20bf6b19e6ca6af0448ea87093d05275b24ef23d7b4b9b17edc93ceec9a4
-  data.tar.gz: 61371d2765af21a3d689931abe71afe4b13942fe4b6c7621115b0d75b1ce0a4d249b28f7ffb40a66e1ab28d5da57f65d3d8814e62372bd13c05c74885cd26507
+  metadata.gz: 7a0a6c9aa9134b043b4a87c8434735f837bca7cfc63520693e3bf49ff4001617351cf92259b73a781187fd22812d65f781130570f753d27dd68d4d6f3bb16fd0
+  data.tar.gz: 14387643b5a8c5b5fc506e00b28b8d5b2664d359bd48797c6180df38f03efa370dd86d46d0a871194328ff80f487458e439733b250dfdad9cf369a253e069c3e

data/lib/noiseless/adapters/execution_modules/http_transport.rb

@@ -14,17 +14,35 @@ module Noiseless
         # DNS failure, transport timeout). These are wrapped into
         # Noiseless::ConnectionError so callers never have to know which HTTP
         # stack is underneath.
-        TRANSPORT_ERRORS = [SystemCallError, SocketError, IOError, Timeout::Error].freeze
-        def initialize(hosts: [], **connection_params)
+        TRANSPORT_ERRORS = [
+          SystemCallError,
+          SocketError,
+          IOError,
+          IO::TimeoutError,
+          Timeout::Error
+        ].freeze
+
+        # Default per-operation IO timeout (seconds) for the search backend.
+        # This is an *idle* timeout: every socket read/write must make progress
+        # within this window. It bounds a stalled/unresponsive backend without
+        # capping the total duration of streaming operations (e.g. bulk import),
+        # since data keeps flowing during those. Override per-connection with
+        # +timeout:+.
+        DEFAULT_TIMEOUT = 5
+
+        def initialize(hosts: [], timeout: DEFAULT_TIMEOUT, **connection_params)
           # Ensure we always have at least one host
           hosts_array = Array(hosts)
           @hosts = hosts_array.empty? ? ["http://localhost:#{default_port}"] : hosts_array
+          @timeout = timeout
           @connection_params = connection_params
 
-          # Initialize HTTP clients for each host
+          # Initialize HTTP clients for each host. The endpoint timeout makes a
+          # stalled backend raise IO::TimeoutError (wrapped below as
+          # ConnectionError) instead of blocking the fiber/reactor indefinitely.
           @clients = {}
           @hosts.each do |host|
-            endpoint = Async::HTTP::Endpoint.parse(host)
+            endpoint = Async::HTTP::Endpoint.parse(host, timeout: @timeout)
             @clients[host] = Async::HTTP::Client.new(endpoint)
           end
 
@@ -77,10 +95,19 @@ module Noiseless
           host = @hosts.sample
           client = @clients[host]
 
-          yield(client)
+          wrap_transport_errors(host: host) { yield(client) }
+        end
+
+        def parse_json_response!(response, error_class: Noiseless::RequestError, context: nil)
+          wrap_transport_errors { super }
+        end
+
+        def wrap_transport_errors(host: nil)
+          yield
         rescue *TRANSPORT_ERRORS => e
+          location = host ? " at #{host}" : ""
           raise Noiseless::ConnectionError,
-                "search backend unreachable at #{host} (#{e.class}: #{e.message})"
+                "search backend unreachable#{location} (#{e.class}: #{e.message})"
         end
 
         def default_headers

data/lib/noiseless/connection_manager.rb

@@ -8,8 +8,8 @@ module Noiseless
     end
 
     # Register a named client statically from YAML (boot-time only)
-    def register(name, adapter:, hosts:)
-      @configs[name.to_sym] = { adapter: adapter, hosts: hosts }
+    def register(name, adapter:, hosts:, timeout: nil)
+      @configs[name.to_sym] = { adapter: adapter, hosts: hosts, timeout: timeout }
     end
 
     # Retrieve a client; defaults to :primary
@@ -19,7 +19,9 @@ module Noiseless
       # Lazy-load the adapter only when actually used
       @clients[name] ||= begin
         config = @configs.fetch(name) { raise "Unknown connection: #{name}" }
-        Adapters.lookup(config[:adapter], hosts: config[:hosts])
+        params = { hosts: config[:hosts] }
+        params[:timeout] = config[:timeout] unless config[:timeout].nil?
+        Adapters.lookup(config[:adapter], **params)
       end
     end
   end

data/lib/noiseless/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Noiseless
-  VERSION = "0.4.0"
+  VERSION = "0.5.0"
 end

data/lib/noiseless.rb

@@ -93,7 +93,7 @@ module Noiseless
     config.connections_config.each do |name, params|
       adapter_name = params[:adapter]
       hosts = params[:hosts] || []
-      connections.register(name, adapter: adapter_name, hosts: hosts)
+      connections.register(name, adapter: adapter_name, hosts: hosts, timeout: params[:timeout])
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: noiseless
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Abdelkader Boudih