legion-transport 1.3.14 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3da418734badd638d77d7743f0558e85b197cf1ed8da2a1ea8149d36da0a16bd
-  data.tar.gz: 90c35ba5ea528cf2ad3f427a5515a4deabfe2e0d90a4992291245ab8607d09f2
+  metadata.gz: a6f0e9a986dae9f69992c0d6c32eb296bbf5fe9bc1c66c746da0ef4eb782f722
+  data.tar.gz: 65d53aebe82cbaba12f6d660d91f1c819235b74663fce117d0405e1e8e780f2b
 SHA512:
-  metadata.gz: d39938df3c7b599a01c030615d1de93827f237806b26174136bdcd5934e7b81416b3c60e432d6090fb26b2d441e20218233c5abf9e1b0914042c936b2da51bee
-  data.tar.gz: 96c1ce706da8cfaa4dd092178c09447a0edf3d4d262f895bf192a6c065805f0bde207880873ccedd9c7799ef56cafc9c400a83030f1266d04400de4ecb53f1ac
+  metadata.gz: 812300aa2df2fea7cfbd7ded068acbac5b44a8487fa83e6df1a680b33e488a51f8db6fa4aeff34aa9fe2829060a71d05572ce209e6211788e113f6e1f41f8d98
+  data.tar.gz: 2ef6853180a0d22dd383f7cdec7e1f2875fe90b4d94cda2fd01c8b1991237de564c4fa5223fb01e6486ef7f2c2da688dea5f7aefa42ca94aa22ef973dc8522b1

data/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Legion::Transport ChangeLog
 
+## [Unreleased]
+
+### Fixed
+- Fix exchange instance cache inheritance: subclasses now correctly read from parent's @instance_cache, preventing boot crash
+- Fix Connection#session: add missing `return` so nil check works correctly
+- Message#publish: use `cached_instance` when available to reuse the exchange instance cache instead of always calling `.new`
+- Message#publish: rescue `Timeout::Error` alongside other network errors so timeouts also spool
+
+### Changed
+- Bump read_timeout from 1 to 3 seconds
+- Bump continuation_timeout from 4000 to 8000 milliseconds
+- Bump session_worker_pool_size from 8 to 16
+
 ## [1.3.14] - 2026-03-24
 
 ### Fixed

data/lib/legion/transport/connection.rb

@@ -91,7 +91,8 @@ module Legion
         end
 
         def session
-          nil if @session.nil?
+          return nil if @session.nil?
+
           @session.value
         end
 

data/lib/legion/transport/exchange.rb

@@ -5,11 +5,39 @@ module Legion
     class Exchange < Legion::Transport::CONNECTOR::Exchange
       include Legion::Transport::Common
 
+      # Thread-local cache of declared exchange instances keyed by class name.
+      # Avoids redundant exchange_declare calls on every publish.
+      @instance_cache = Concurrent::ThreadLocalVar.new { {} }
+
+      class << self
+        def instance_cache
+          Legion::Transport::Exchange.instance_variable_get(:@instance_cache)
+        end
+
+        def cached_instance
+          cache = instance_cache.value
+          inst = cache[name]
+          return inst if inst&.channel&.open?
+
+          cache.delete(name)
+          nil
+        end
+
+        def cache_instance(inst)
+          instance_cache.value[name] = inst
+        end
+
+        def clear_cache
+          instance_cache.value.clear
+        end
+      end
+
       def initialize(exchange = exchange_name, options = {})
         @options = options
         @explicit_channel = @options.delete(:channel)
         @type = options[:type] || default_type
         super(channel, @type, exchange, options_builder(default_options, exchange_options, @options))
+        self.class.cache_instance(self) if self.class.respond_to?(:cache_instance)
       rescue Legion::Transport::CONNECTOR::PreconditionFailed, Legion::Transport::CONNECTOR::ChannelAlreadyClosed
         raise unless @retries.nil?
 

data/lib/legion/transport/message.rb

@@ -13,7 +13,14 @@ module Legion
       def publish(options = @options)
         raise unless @valid
 
-        exchange_dest = exchange.respond_to?(:new) ? exchange.new : exchange
+        ex_class = exchange
+        exchange_dest = if ex_class.respond_to?(:cached_instance)
+                          ex_class.cached_instance || ex_class.new
+                        elsif ex_class.respond_to?(:new)
+                          ex_class.new
+                        else
+                          ex_class
+                        end
         exchange_dest.publish(encode_message,
                               routing_key:      routing_key || '',
                               content_type:     options[:content_type] || content_type,
@@ -30,7 +37,7 @@ module Legion
         ex_name = exchange_dest.respond_to?(:name) ? exchange_dest.name : exchange_dest.to_s
         Legion::Logging.debug "Published to exchange=#{ex_name} routing_key=#{routing_key || ''} class=#{self.class.name}" if defined?(Legion::Logging)
       rescue Bunny::ConnectionClosedError, Bunny::ChannelAlreadyClosed, Bunny::ChannelError,
-             Bunny::NetworkErrorWrapper, IOError => e
+             Bunny::NetworkErrorWrapper, IOError, Timeout::Error => e
         spool_message(e)
       end
 

data/lib/legion/transport/settings.rb

@@ -15,10 +15,10 @@ module Legion
         extra_hosts   = existing[:hosts] || []
 
         {
-          read_timeout:              1,
+          read_timeout:              3,
           heartbeat:                 (ENV['transport.connection.heartbeat'] || 30).to_i,
           automatically_recover:     true,
-          continuation_timeout:      4000,
+          continuation_timeout:      8000,
           network_recovery_interval: (ENV['transport.connection.recovery_interval'] || 2).to_i,
           connection_timeout:        (ENV['transport.connection.connection_timeout'] || 10).to_i,
           frame_max:                 65_536,
@@ -65,7 +65,7 @@ module Legion
       def self.channel
         {
           default_worker_pool_size: ENV['transport.channel.default_worker_pool_size'] || 1,
-          session_worker_pool_size: ENV['transport.channel.session_worker_pool_size'] || 8
+          session_worker_pool_size: ENV['transport.channel.session_worker_pool_size'] || 16
         }
       end
 

data/lib/legion/transport/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module Transport
-    VERSION = '1.3.14'
+    VERSION = '1.4.0'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-transport
 version: !ruby/object:Gem::Version
-  version: 1.3.14
+  version: 1.4.0
 platform: ruby
 authors:
 - Esity