omq 0.17.6 → 0.17.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b990bcb03502bd0cc09d92cf0c402dfeca1602faec96ecfce85500f0c1d2aa7b
-  data.tar.gz: 129e85fa71560045d6bba7d1fbb3c21c8c12cba112c840e70802fc81606fe23b
+  metadata.gz: 684f878ff9a1731c4820ec6a6ed52393b727fa046b3d8ff74dfbb4fd50e20e48
+  data.tar.gz: f5ffad6b1c827949a3c17088536af9a7b7e3f329231767b299cb5bd5a0f73628
 SHA512:
-  metadata.gz: e729dc6c1ff1db46cba68feeb0bd017cf09049a1a65b7e908b54cbcf398851ecb04c4ee5dae8ee35244061638e85c1314dcee9cfa458026c30c7fe27543689ee
-  data.tar.gz: ea9d7aa704da50ff2cb00ecd254888dfb613e27d65a3c2e72bf5f6330f458060417da18f7c84f56902b3d53ea77c89d98a71837d33dafe2cbc0b6b03fb2d9cc5
+  metadata.gz: ec98f8caf95091396b6a7f4e3ed01480700c4ba7fb14d3c85dc20572e6fd2cf3160f1f081f5390d79834d8b9db8271a6f8104c34400562c874fbf61c2763cfab
+  data.tar.gz: 15a546c0717ab0440aeb010bd9fca87f01f63bffe4eba91fa852b2a48bb99864994c366778114e116463c715dbab11e218256602c662d394dc6a17b5657e683f

data/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Changelog
 
+## 0.17.8 — 2026-04-10
+
+### Fixed
+
+- **Linger drain missed in-flight messages.** `RoundRobin#send_queues_drained?`
+  now tracks an `@in_flight` counter for messages dequeued by pump fibers but
+  not yet written. Previously, linger could tear down connections while pumps
+  still held unwritten batches, dropping messages silently.
+
+## 0.17.7 — 2026-04-10
+
+### Changed
+
+- **Message parts coerced via `#to_s`.** `#frozen_binary` now calls
+  `#to_s` instead of `#to_str`, so `nil` becomes an empty frame and
+  integers/symbols are converted automatically. A cached `EMPTY_PART`
+  avoids allocations for nil parts.
+
+- **Reduced allocations on hot paths.** `freeze_message` short-circuits
+  when all parts are already frozen binary (zero-alloc fast path).
+  `write_batch` passes the batch directly instead of `.map`-ing through
+  `transform_send` — only REQ overrides the transform and it never
+  batches. Up to +55% throughput on small messages (PUSH/PULL IPC 64B).
+
 ## 0.17.6 — 2026-04-10
 
 ### Fixed

data/lib/omq/routing/round_robin.rb

@@ -20,9 +20,10 @@ module OMQ
     #
     module RoundRobin
       # @return [Boolean] true when the shared send queue is empty
+      #   and no pump fiber is mid-write with a dequeued batch.
       #
       def send_queues_drained?
-        @send_queue.empty?
+        @send_queue.empty? && @in_flight == 0
       end
 
       private
@@ -36,6 +37,7 @@ module OMQ
         @send_queue      = Routing.build_queue(engine.options.send_hwm, :block)
         @direct_pipe     = nil
         @conn_send_tasks = {}  # conn => send pump task
+        @in_flight       = 0   # messages dequeued but not yet written
       end
 
 
@@ -126,7 +128,12 @@ module OMQ
           loop do
             batch = [@send_queue.dequeue]
             drain_send_queue_capped(batch)
-            write_batch(conn, batch)
+            @in_flight += batch.size
+            begin
+              write_batch(conn, batch)
+            ensure
+              @in_flight -= batch.size
+            end
             batch.each { |parts| @engine.emit_verbose_monitor_event(:message_sent, parts: parts) }
             Async::Task.current.yield
           end
@@ -154,10 +161,7 @@ module OMQ
         if batch.size == 1
           conn.send_message(transform_send(batch[0]))
         else
-          # Single mutex acquisition for the whole batch (up to
-          # BATCH_MSG_CAP messages). transform_send is identity for
-          # most routings and only allocates a new parts array for REQ.
-          conn.write_messages(batch.map { |parts| transform_send(parts) })
+          conn.write_messages(batch)
           conn.flush
         end
       end

data/lib/omq/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OMQ
-  VERSION = "0.17.6"
+  VERSION = "0.17.8"
 end

data/lib/omq/writable.rb

@@ -39,17 +39,25 @@ module OMQ
     def freeze_message(message)
       parts = message.is_a?(Array) ? message : [message]
       raise ArgumentError, "message has no parts" if parts.empty?
+
+      # Fast path: skip map when all parts are already frozen binary.
       if parts.frozen?
+        return parts if parts.all? { |p| p.is_a?(String) && p.frozen? && p.encoding == Encoding::BINARY }
         parts = parts.map { |p| frozen_binary(p) }
       else
-        parts.map! { |p| frozen_binary(p) }
+        unless parts.all? { |p| p.is_a?(String) && p.frozen? && p.encoding == Encoding::BINARY }
+          parts.map! { |p| frozen_binary(p) }
+        end
       end
       parts.freeze
     end
 
 
-    def frozen_binary(str)
-      s = str.to_str
+    EMPTY_PART = "".b.freeze
+
+    def frozen_binary(obj)
+      return EMPTY_PART if obj.nil?
+      s = obj.to_s
       return s if s.frozen? && s.encoding == Encoding::BINARY
       s.b.freeze
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: omq
 version: !ruby/object:Gem::Version
-  version: 0.17.6
+  version: 0.17.8
 platform: ruby
 authors:
 - Patrik Wenger