karafka-rdkafka 0.27.2 → 0.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 28b25d5553da8103cf3ba2840c57d954bc36a957004a01e31a7a4ace76b82382
-  data.tar.gz: ee834fb05a4afa6ac3bb31d2dcf8735779f6bcd9c7e4b910d95e22ecae74b3fc
+  metadata.gz: 04f165f0ef5a6a2a91906f9bfd8ea143acf5197f7e13b4d1d4d1625bce4e792c
+  data.tar.gz: 6c03af6f233ed78de1e2d02482170cf4e2d50d98f54399dd555ce08c49bfacc8
 SHA512:
-  metadata.gz: d4fe487cab97e1cf1ea5c275a682feee20247d1e20517d257c1a03383d5fb9bcb926404626123e3d0c7e05d9832fa7675c37b0b5f6b0ff484c277324a432c66f
-  data.tar.gz: 223c61df2cb43ad07df271caf29c535e8daa13f49602a21cfea5ef8ddb030e6378f23fe698f15025a673083fd486b749c04f3078edb6f69d14fa0339cd55658f
+  metadata.gz: 622a269eaa8f11e02988ca1de4c26a07fe062d708c5c4a357c19a07486edd34b67c7c1b1e35962ed60125ec08ccc4a10692b9d46e0dc997c199d33e1b20d3195
+  data.tar.gz: c668fb57232d3dc6d53e0065db03470ed4a81d3d4820097d018bc5a71991e042e66d12b2051138d1c85955acb3b985f986a7477a610cc94ebdde5fbf4ac7ef2e

data/CHANGELOG.md

@@ -1,5 +1,38 @@
 # Rdkafka Changelog
 
+## v0.28.0
+- [Enhancement] Bump librdkafka to `2.14.2`. Maintenance release: fixes duplicate groups in `ListConsumerGroups` when multiple brokers return the same group, a data race in timers, and bumps bundled OpenSSL/libcurl/zstd/zlib/cJSON dependencies (several CVEs).
+- [Enhancement] Add `Consumer#metadata` and `Producer#metadata`, mirroring `Admin#metadata`, so cluster/topic metadata can be fetched from an existing consumer or producer handle without opening a dedicated admin connection.
+- [Enhancement] Name the failing partition and topic in the `RdkafkaError` raised for per-partition `list_offsets` errors (previously a bare error code), preserving the per-partition context the pre-batching `Consumer#lag` watermark errors carried.
+- [Enhancement] Add `Consumer#list_offsets`, mirroring `Admin#list_offsets`, so batched offset queries (one `ListOffsets` request carrying all requested partitions, fanned out to the partition leaders by librdkafka) can be issued on an existing consumer handle without opening a dedicated admin connection, and rebuild `Consumer#lag` on top of it: lag is now computed from a single batched end-offsets query instead of one blocking `query_watermark_offsets` broker roundtrip per partition. To receive the results, consumer clients now register the librdkafka background event callback, so librdkafka spawns its internally-managed background thread for consumers as well. `Consumer#lag` keeps its exact pre-batching semantics: it forwards the consumer's configured `isolation.level` to the batched query (librdkafka resolves the per-partition watermark query with that level, so end offsets stay LSO-based for the default `read_committed`), surfaces a timeout of the batched query as an `RdkafkaError` (`timed_out`), and now also raises `ClosedConsumerError` when called on a closed consumer.
+- [Enhancement] Extract the admin background-event result handlers into one class per operation under `lib/rdkafka/callbacks/` (`CreateTopicHandler`, `DescribeConfigsHandler`, etc., all subclasses of `Callbacks::BaseHandler`). `Callbacks::BackgroundEventCallback` is now a thin dispatcher that maps the event type to its handler and destroys the event. Purely internal reorganization (`Rdkafka::Callbacks` is private) with no behavior or API change.
+- [Enhancement] Expose `replicas` and `isrs` (in-sync replica broker ids) on each partition in topic metadata (`Metadata#topics` partition hashes). The base struct `#to_h` skips FFI pointer members, so these two arrays were dropped entirely and the partition replica assignment was unavailable to callers (e.g. for planning replication-factor changes). `PartitionMetadata#to_h` now dereferences both pointers into arrays of broker ids.
+- [Enhancement] Reuse per-thread scratch pointers in `Consumer::Headers.from_native` instead of allocating them for every consumed message. Previously each message paid one native pointer allocation just to check for headers and three more when headers were present; now the scratch pointers are allocated once per thread/fiber and reused, removing all per-message native scratch allocations from the consumer hot path.
+- [Enhancement] Remove the unused `DeliveryHandle` `:topic_name` struct field and the per-message allocation that populated it. The delivery callback copied the topic name into a native `FFI::MemoryPointer` on every delivered message, retained for the lifetime of the handle, yet nothing ever read it: the topic is already available via `DeliveryHandle#topic` (a Ruby attribute set during `produce`) and `DeliveryReport#topic_name`, both of which work exactly as before.
+- [Fix] Stop `poll_batch`/`poll_batch_nb` from discarding a whole batch when one message fails to build. If building a `Consumer::Message` raised (e.g. a header-read `RdkafkaError`), the exception propagated out of the batch loop and the `ensure` destroyed every remaining message, so all the messages already built in that batch were lost even though their offsets had been stored - permanent silent loss of up to `max_items` messages. A per-message build error is now surfaced inline as an `RdkafkaError` in the returned array (the same way native error events already are), so the rest of the batch is preserved and the failure is visible to the caller. (Single-message `poll`/`poll_nb` still raise as before.)
+- [Fix] Add the missing `closed_consumer_check` to `Consumer#position`. Every sibling offset method guards against a closed consumer, but `position` did not: with an explicit `list` it raised `ClosedInnerError` instead of `ClosedConsumerError`, and with no `list` it raised `ClosedConsumerError` naming `assignment` rather than `position`. It now raises a consistent `ClosedConsumerError` for `position`.
+- [Fix] Stop leaking the native `rd_kafka_topic_conf_t` in `Producer#set_topic_config` when a per-topic config value is rejected. The conf is built with `rd_kafka_topic_conf_new` and only handed to `rd_kafka_topic_new` afterwards; if `rd_kafka_topic_conf_set` returned a non-`:config_ok` result the method raised before `rd_kafka_topic_new` was ever called, so librdkafka never took ownership of the conf and it leaked (repeatable per produce with an invalid `topic_config:`). The conf is now destroyed before raising. The `rd_kafka_topic_new` path is unchanged: librdkafka frees the conf there on both success and failure, so it is never destroyed by us.
+- [Fix] Raise instead of silently dropping a rejected `incremental_alter_configs` entry. `rd_kafka_ConfigResource_add_incremental_config` returns an `rd_kafka_error_t` for an invalid op_type, an empty/nil name, or a nil value on a non-delete op; the result was ignored, so the entry was dropped, the alter request still reported success, and the error object leaked. The error is now surfaced as an `RdkafkaError` (raised before the request is sent) and the native error object is freed.
+- [Fix] Let `PartitionsCountCache` adopt a lower partition count once the cached entry has expired. The cache prioritizes higher counts (partition counts only grow during normal operation), but it did so unconditionally: after the TTL expired it fetched the true lower count, discarded it, and re-armed the TTL on the stale higher count - permanently. A topic recreated with fewer partitions then made `produce` (with a partition key) fail with `unknown_partition` for the dropped partitions until process restart. A lower value is now adopted on the first refresh after expiry, while still being ignored within the TTL window so a transient or racy lower read cannot clobber a correct higher count.
+- [Fix] Make the `Consumer` GC finalizer close the consumer and destroy its consumer queue, not just the native client. The finalizer used the generic `NativeKafka` one, which went straight to `rd_kafka_destroy` and skipped `rd_kafka_consumer_close` and `rd_kafka_queue_destroy`. A consumer that used `poll_batch` (which takes a consumer-queue reference) and was then garbage-collected without an explicit `close` left that reference dangling, which could make `rd_kafka_destroy` block inside the finalizer (process hang at GC/shutdown) or leak the handle. The consumer now installs its own finalizer that mirrors `#close` (the queue pointer is shared with the finalizer via a holder so it never captures the consumer and prevents collection).
+- [Fix] Stop admin operations from leaking when their arguments are rejected. `describe_configs` and `incremental_alter_configs` allocated the background queue and AdminOptions and registered their handle before parsing input, so a missing key raised a `KeyError` out of the building loop that orphaned the handle in the process-global registry forever (and leaked the queue, AdminOptions and any `ConfigResource`s already built); `list_offsets` likewise allocated its native topic-partition list before validating the offset specs. All three now parse and validate input up front, so a bad argument raises with nothing allocated.
+- [Fix] Destroy the native topic-partition list in `TopicPartitionList#to_native_tpl` when population fails partway. The list is allocated with `rd_kafka_topic_partition_list_new` and only handed back (for the caller to destroy) on success; if building it raised - e.g. a non-string metadata value or an offset FFI cannot coerce to int64 - the half-built list leaked, since destruction is fully manual (the previous doc comment claiming GC handled it was wrong). It is now destroyed before the error propagates.
+- [Fix] Allocate the admin result-count out-parameter as `:size_t` instead of `:int32`. Every librdkafka `rd_kafka_*_result_*(result, size_t *cntp)` accessor writes a full native `size_t` (8 bytes on 64-bit), but the count pointer was a 4-byte `FFI::MemoryPointer.new(:int32)` across the create/delete topic, create partitions, create/delete ACL, describe/incremental-alter configs and config-synonyms paths - a 4-byte heap overflow on every admin result parse (benign on little-endian, where the low word still reads the correct count, but undefined behavior). Now uses `:size_t`, matching the already-correct list-offsets and `get_err_descs` paths.
+- [Fix] Destroy admin API background events after processing. librdkafka requires the application to destroy each background event, but `rd_kafka_event_destroy` was never called (nor even bound), so every admin operation leaked its entire result event with all result arrays and strings. Reports are now built inside the callback (copying event-owned memory into Ruby objects) before the event is destroyed, and `DescribeConfigsReport`/`IncrementalAlterConfigsReport` no longer destroy the event-owned ConfigResource array, which also fixes a double free on repeated `wait` calls on the same handle. As part of this, the internal FFI struct fields on admin operation handles (e.g. `handle[:error_string]`, `handle[:result_name]`, `handle[:config_entries]`, `handle[:response_string]`, `handle[:matching_acls]`) were removed; they were never part of the public API (use `handle.wait` and the returned report objects, whose interfaces are unchanged).
+- [Fix] Resolve admin operation handles from the event error when an admin operation fails at the operation level (e.g. brokers unreachable, or the client closed with the request in flight). librdkafka delivers such failures as a result event with the error set and an empty results array, but the create topic, delete topic, create partitions, delete groups, create ACL and delete ACL handlers indexed `results[0]` unconditionally. That raised inside the background event callback, so the handle was never unlocked and `wait` blocked until its own timeout and raised `WaitTimeoutError`, discarding the real error. These handlers now check the event error first and resolve the handle with the actual error code (the describe configs, incremental alter configs, describe ACL and list offsets handlers already did).
+- [Fix] Stop leaking the native `rd_kafka_conf_t` when client creation fails. `native_config` raised `ConfigError` mid-build (e.g. on an invalid option) without destroying the conf, and `native_kafka` raised `ClientCreationError` on a null `rd_kafka_new` without destroying it either. Both paths now call `rd_kafka_conf_destroy` before re-raising (librdkafka keeps app ownership of the conf on `rd_kafka_new` failure, so this is safe). Multi-KB leak per failed creation, relevant for supervisors retrying client creation on transient SASL/SSL misconfig.
+- [Fix] Stop `Metadata` from leaking the native metadata struct (and a topic reference) on every retried fetch. `retry` restarts the `begin` block without running its `ensure`, so each retried attempt reassigned the pointers and only the last attempt's `rd_kafka_metadata` struct was ever destroyed; up to `METADATA_MAX_RETRIES` whole-cluster structs leaked per call (the `leader_not_available` case is routine during topic creation/leader election). Each attempt now frees its own native resources, and a failed fetch that never allocated a struct no longer calls `rd_kafka_metadata_destroy` on a NULL pointer.
+- [Fix] Free the librdkafka-allocated string in `Consumer#cluster_id` and `Consumer#member_id` (previously copied via a `:string` binding but never freed) and fix the `rd_kafka_clusterid` arity to pass `timeout_ms`. `Consumer#cluster_id` now accepts a `timeout_ms` (default `Defaults::CONSUMER_CLUSTER_ID_TIMEOUT_MS`).
+- [Fix] Guard the message delivery callback so a raising user `delivery_callback` can no longer skip the handle unlock or crash the producer. `DeliveryCallback` invoked the user callback and only then unlocked the handle, with no rescue; if the callback raised, the handle stayed pending (so `wait` blocked until its timeout and raised `WaitTimeoutError` for a message that was actually delivered) and the exception unwound out of the FFI callback on librdkafka's polling thread (`abort_on_exception = true`), taking down the whole process. The user callback is now wrapped so exceptions are logged and swallowed (matching the rebalance callback) and the handle is always unlocked in an `ensure`.
+- [Fix] Stop `Producer#produce` from orphaning the delivery handle in the process-global registry when it fails after registering it. The handle was only removed on a non-zero `rd_kafka_producev` return, so any exception between registration and that check (a concurrent `close` making `with_inner` raise `ClosedInnerError`, or a header value whose `#to_s` raises) leaked the handle forever - it survives producer close and accumulates in apps that recreate/close producers. `produce` now removes the handle on any such failure before re-raising.
+- [Fix] Attach `rd_kafka_query_watermark_offsets` with `blocking: true` so it releases the GVL during its broker round-trip. It was the only synchronous network call bound without the flag, so `Consumer#query_watermark_offsets` (and `Consumer#lag`, which calls it once per partition) froze every other Ruby thread in the process - including producer polling threads - for up to `timeout_ms`. Matches the neighboring `rd_kafka_offsets_for_times` binding.
+- [Fix] Stabilize the flaky `Consumer#lag` "calculates the consumer lag" spec on overloaded CI. The manual `consumer.commit` could raise `no_offset` when the default 5s background auto-commit had already committed the stored offsets, or when the auto offset store had not yet caught up with `poll`. The spec now raises `auto.commit.interval.ms` to 60s (matching the existing `#seek`/pause specs) and lets the offset store settle before committing. Backported from rdkafka-ruby (#912).
+- [Fix] Fix the NULL background-queue cleanup branches in `Admin#delete_group`, `Admin#delete_acl` and `Admin#describe_acl`, which referenced undefined local variables (`delete_topic_ptr`/`new_acl_ptr`). When `rd_kafka_queue_get_background` returned NULL, those branches raised a `NameError` instead of the intended `ConfigError` and leaked the already-allocated native request object (the `rd_kafka_DeleteGroup_t` / ACL binding filter), and `delete_group` additionally called the wrong destructor. They now destroy the correct object and raise `ConfigError`.
+- [Fix] Forward `broker_message` and `instance_name` through `RdkafkaError.build`. On the `rd_kafka_error_t` pointer path `build` called `build_from_c` without passing either, so a caller-supplied broker message was discarded (falling back to `rd_kafka_err2str`) and the instance name was lost; the `Bindings::Message` path also dropped `instance_name`. Both are now forwarded, and `build_from_c` accepts `instance_name`.
+- [Fix] Synchronize `AbstractHandle::REGISTRY` mutations. The handle registry is a plain Hash mutated from producing/consuming threads and the background polling thread (which removes handles from FFI callbacks). That is effectively safe on MRI under the GVL but not on JRuby, where a lost write could leave a handle unregistered (never unlocked, so `wait` times out for a delivered message) or never removed (permanent leak). `register`/`remove` now guard the Hash with a mutex.
+- [Fix] Stop the `Metadata` retry loop from clobbering the request timeout and blocking for minutes. Each retry overwrote the per-request timeout with the exponential backoff value, so the first retries ran with a far-too-short request timeout (e.g. 200ms vs the 2,000ms default, near-guaranteeing another timeout) while later ones inflated it to ~100s; the cumulative sleeps alone reached ~204s. The request timeout is now left unchanged across retries, the per-attempt backoff is capped at `Defaults::METADATA_RETRY_BACKOFF_MAX_MS` (1,000ms), and the loop is bounded by a wall-clock budget (`Defaults::METADATA_RETRY_BUDGET_MS`, 5,000ms) after a floor of `Defaults::METADATA_MIN_ATTEMPTS` (3) tries. This keeps a synchronous metadata fetch from blocking for minutes while still giving a slow broker a few attempts: fast-recovering clusters retry quickly within the budget, an unresponsive one fails in ~5s (a bit over if its requests each consume the full timeout), and at least 3 attempts always happen.
+- [Fix] Cache the partition count for a missing topic so `produce` with a `partition_key` to a not-yet-created topic no longer runs a blocking metadata query on every message. The `unknown_topic_or_part` result was resolved to `RD_KAFKA_PARTITION_UA` outside the partition-count cache, so nothing was stored and each call performed a fresh synchronous metadata RPC (the promised negative cache did not exist). The miss is now resolved inside the cache block, so it is cached like any other value and reused for the cache TTL.
+
 ## 0.27.2 (2026-05-21)
 - [Enhancement] `poll_batch` and `poll_batch_nb` now return error events inline as `RdkafkaError` objects rather than raising on the first error. The return type is `Array<Message, RdkafkaError>` and callers are responsible for handling errors in the result.
 

data/README.md

@@ -63,6 +63,7 @@ Contributions should generally be made to the upstream [rdkafka-ruby repository]
 
 | rdkafka-ruby | librdkafka | patches |
 |-|-|-|
+| 0.28.x (2026-07-12) | 2.14.2 (2026-06-03)  | yes |
 | 0.27.x (2026-05-08) | 2.14.1 (2026-04-15)  | yes |
 | 0.26.x (2026-04-11) | 2.14.0 (2026-04-01)  | yes |
 | 0.25.x (2026-04-02) | 2.13.2 (2026-03-02)  | yes |

data/docker-compose-ssl.yml

@@ -1,7 +1,7 @@
 services:
   kafka:
     container_name: kafka
-    image: confluentinc/cp-kafka:8.2.0
+    image: confluentinc/cp-kafka:8.3.0
     ports:
       - 9092:9092 # Support PLAINTEXT so we can run one docker setup for SSL and PLAINTEXT
       - 9093:9093

data/docker-compose.yml

@@ -1,7 +1,7 @@
 services:
   kafka:
     container_name: kafka
-    image: confluentinc/cp-kafka:8.2.0
+    image: confluentinc/cp-kafka:8.3.0
 
     ports:
       - 9092:9092

data/karafka-rdkafka.gemspec

@@ -19,6 +19,7 @@ Gem::Specification.new do |gem|
     next true if file.start_with?(".")
     next true if file.start_with?("spec/")
     next true if file.start_with?("ext/README.md")
+    next true if file.start_with?("Gemfile")
 
     false
   end

data/lib/rdkafka/abstract_handle.rb

@@ -14,9 +14,16 @@ module Rdkafka
 
     # Registry for registering all the handles.
     REGISTRY = {}
+    # Guards REGISTRY mutations. The registry is shared across producing/consuming threads and the
+    # background polling thread (which removes handles from FFI callbacks), so the Hash must not be
+    # mutated concurrently. This is effectively safe on MRI under the GVL but not on JRuby, where a
+    # lost write could leave a handle unregistered (never unlocked -> `wait` times out for a
+    # delivered message) or never removed (permanent leak).
+    REGISTRY_MUTEX = Mutex.new
     # Default wait timeout is 31 years
     MAX_WAIT_TIMEOUT_FOREVER = 10_000_000_000
 
+    private_constant :REGISTRY_MUTEX
     private_constant :MAX_WAIT_TIMEOUT_FOREVER
 
     class << self
@@ -25,17 +32,28 @@ module Rdkafka
       # @param handle [AbstractHandle] any handle we want to register
       def register(handle)
         address = handle.to_ptr.address
-        REGISTRY[address] = handle
+        REGISTRY_MUTEX.synchronize { REGISTRY[address] = handle }
       end
 
       # Removes handle from the register based on the handle address
       #
       # @param address [Integer] address of the registered handle we want to remove
       def remove(address)
-        REGISTRY.delete(address)
+        REGISTRY_MUTEX.synchronize { REGISTRY.delete(address) }
       end
     end
 
+    # Operation result prepared by the background event callback. Background events are
+    # destroyed as soon as the callback returns, so anything the waiting thread needs has to be
+    # copied out of event-owned memory into Ruby objects and stored here before `#unlock` runs.
+    # When result parsing fails, holds the captured exception so it can be re-raised on the
+    # waiting thread.
+    attr_accessor :result
+
+    # Broker-provided error message copied out of event-owned memory by the background event
+    # callback, used by `#raise_error`
+    attr_accessor :broker_message
+
     def initialize
       @mutex = Thread::Mutex.new
       @resource = Thread::ConditionVariable.new
@@ -118,13 +136,28 @@ module Rdkafka
     end
 
     # @return [Object] operation-specific result
+    #
+    # Defaults to the result prepared by the background event callback (see
+    # {#prepared_result}). Subclasses backed by a synchronous callback that writes the result
+    # into struct fields (e.g. {Producer::DeliveryHandle}) override this.
     def create_result
-      raise "Must be implemented by subclass!"
+      prepared_result
+    end
+
+    # Returns the operation result prepared by the background event callback, re-raising an
+    # exception that was captured while the event memory was still readable
+    #
+    # @return [Object] operation-specific result
+    # @raise [RdkafkaError] when result parsing in the event callback failed
+    def prepared_result
+      raise(result) if result.is_a?(Exception)
+
+      result
     end
 
     # Allow subclasses to override
     def raise_error
-      RdkafkaError.validate!(self[:response])
+      RdkafkaError.validate!(self[:response], broker_message: broker_message)
     end
 
     # Error that is raised when waiting for the handle to complete

data/lib/rdkafka/admin/config_binding_result.rb

@@ -17,7 +17,7 @@ module Rdkafka
         @synonyms = []
 
         # The code below builds up the config synonyms using same config binding
-        pointer_to_size_t = FFI::MemoryPointer.new(:int32)
+        pointer_to_size_t = FFI::MemoryPointer.new(:size_t)
         synonym_ptr = Bindings.rd_kafka_ConfigEntry_synonyms(config_ptr, pointer_to_size_t)
         synonyms_ptr = synonym_ptr.read_array_of_pointer(pointer_to_size_t.read_int)
 

data/lib/rdkafka/admin/create_acl_handle.rb

@@ -5,25 +5,19 @@ module Rdkafka
     # Handle for create ACL operation
     class CreateAclHandle < AbstractHandle
       layout :pending, :bool,
-        :response, :int,
-        :response_string, :pointer
+        :response, :int
 
       # @return [String] the name of the operation
       def operation_name
         "create acl"
       end
 
-      # @return [CreateAclReport] instance with rdkafka_response value as 0 and rdkafka_response_string value as empty string if the acl creation was successful
-      def create_result
-        CreateAclReport.new(rdkafka_response: self[:response], rdkafka_response_string: self[:response_string])
-      end
-
       # Raises an error if the operation failed
       # @raise [RdkafkaError]
       def raise_error
         raise RdkafkaError.new(
           self[:response],
-          broker_message: self[:response_string].read_string
+          broker_message: broker_message
         )
       end
     end

data/lib/rdkafka/admin/create_partitions_handle.rb

@@ -3,31 +3,12 @@ module Rdkafka
     # Handle for create partitions operation
     class CreatePartitionsHandle < AbstractHandle
       layout :pending, :bool,
-        :response, :int,
-        :error_string, :pointer,
-        :result_name, :pointer
+        :response, :int
 
       # @return [String] the name of the operation
       def operation_name
         "create partitions"
       end
-
-      # @return [Boolean] whether the create topic was successful
-      def create_result
-        CreatePartitionsReport.new(self[:error_string], self[:result_name])
-      end
-
-      # Raises an error if the operation failed
-      # @raise [RdkafkaError]
-      def raise_error
-        RdkafkaError.validate!(
-          self[:response],
-          broker_message: CreatePartitionsReport.new(
-            self[:error_string],
-            self[:result_name]
-          ).error_string
-        )
-      end
     end
   end
 end

data/lib/rdkafka/admin/create_topic_handle.rb

@@ -5,31 +5,12 @@ module Rdkafka
     # Handle for create topic operation
     class CreateTopicHandle < AbstractHandle
       layout :pending, :bool,
-        :response, :int,
-        :error_string, :pointer,
-        :result_name, :pointer
+        :response, :int
 
       # @return [String] the name of the operation
       def operation_name
         "create topic"
       end
-
-      # @return [Boolean] whether the create topic was successful
-      def create_result
-        CreateTopicReport.new(self[:error_string], self[:result_name])
-      end
-
-      # Raises an error if the operation failed
-      # @raise [RdkafkaError]
-      def raise_error
-        RdkafkaError.validate!(
-          self[:response],
-          broker_message: CreateTopicReport.new(
-            self[:error_string],
-            self[:result_name]
-          ).error_string
-        )
-      end
     end
   end
 end

data/lib/rdkafka/admin/delete_acl_handle.rb

@@ -5,27 +5,19 @@ module Rdkafka
     # Handle for delete ACL operation
     class DeleteAclHandle < AbstractHandle
       layout :pending, :bool,
-        :response, :int,
-        :response_string, :pointer,
-        :matching_acls, :pointer,
-        :matching_acls_count, :int
+        :response, :int
 
       # @return [String] the name of the operation
       def operation_name
         "delete acl"
       end
 
-      # @return [DeleteAclReport] instance with an array of matching_acls
-      def create_result
-        DeleteAclReport.new(matching_acls: self[:matching_acls], matching_acls_count: self[:matching_acls_count])
-      end
-
       # Raises an error if the operation failed
       # @raise [RdkafkaError]
       def raise_error
         raise RdkafkaError.new(
           self[:response],
-          broker_message: self[:response_string].read_string
+          broker_message: broker_message
         )
       end
     end

data/lib/rdkafka/admin/delete_groups_handle.rb

@@ -4,28 +4,20 @@ module Rdkafka
   class Admin
     # Handle for delete groups operation
     class DeleteGroupsHandle < AbstractHandle
-      layout :pending, :bool, # TODO: ???
-        :response, :int,
-        :error_string, :pointer,
-        :result_name, :pointer
+      layout :pending, :bool,
+        :response, :int
 
       # @return [String] the name of the operation
       def operation_name
         "delete groups"
       end
 
-      # Creates the result report
-      # @return [DeleteGroupsReport]
-      def create_result
-        DeleteGroupsReport.new(self[:error_string], self[:result_name])
-      end
-
       # Raises an error if the operation failed
       # @raise [RdkafkaError]
       def raise_error
         raise RdkafkaError.new(
           self[:response],
-          broker_message: create_result.error_string
+          broker_message: broker_message
         )
       end
     end

data/lib/rdkafka/admin/delete_topic_handle.rb

@@ -5,31 +5,12 @@ module Rdkafka
     # Handle for delete topic operation
     class DeleteTopicHandle < AbstractHandle
       layout :pending, :bool,
-        :response, :int,
-        :error_string, :pointer,
-        :result_name, :pointer
+        :response, :int
 
       # @return [String] the name of the operation
       def operation_name
         "delete topic"
       end
-
-      # @return [Boolean] whether the delete topic was successful
-      def create_result
-        DeleteTopicReport.new(self[:error_string], self[:result_name])
-      end
-
-      # Raises an error if the operation failed
-      # @raise [RdkafkaError]
-      def raise_error
-        RdkafkaError.validate!(
-          self[:response],
-          broker_message: DeleteTopicReport.new(
-            self[:error_string],
-            self[:result_name]
-          ).error_string
-        )
-      end
     end
   end
 end

data/lib/rdkafka/admin/describe_acl_handle.rb

@@ -5,27 +5,19 @@ module Rdkafka
     # Handle for describe ACL operation
     class DescribeAclHandle < AbstractHandle
       layout :pending, :bool,
-        :response, :int,
-        :response_string, :pointer,
-        :acls, :pointer,
-        :acls_count, :int
+        :response, :int
 
       # @return [String] the name of the operation.
       def operation_name
         "describe acl"
       end
 
-      # @return [DescribeAclReport] instance with an array of acls that matches the request filters.
-      def create_result
-        DescribeAclReport.new(acls: self[:acls], acls_count: self[:acls_count])
-      end
-
       # Raises an error if the operation failed
       # @raise [RdkafkaError]
       def raise_error
         raise RdkafkaError.new(
           self[:response],
-          broker_message: self[:response_string].read_string
+          broker_message: broker_message
         )
       end
     end

data/lib/rdkafka/admin/describe_configs_handle.rb

@@ -5,30 +5,19 @@ module Rdkafka
     # Handle for describe configs operation
     class DescribeConfigsHandle < AbstractHandle
       layout :pending, :bool,
-        :response, :int,
-        :response_string, :pointer,
-        :config_entries, :pointer,
-        :entry_count, :int
+        :response, :int
 
       # @return [String] the name of the operation.
       def operation_name
         "describe configs"
       end
 
-      # @return [DescribeAclReport] instance with an array of acls that matches the request filters.
-      def create_result
-        DescribeConfigsReport.new(
-          config_entries: self[:config_entries],
-          entry_count: self[:entry_count]
-        )
-      end
-
       # Raises an error if the operation failed
       # @raise [RdkafkaError]
       def raise_error
         raise RdkafkaError.new(
           self[:response],
-          broker_message: self[:response_string].read_string
+          broker_message: broker_message
         )
       end
     end

data/lib/rdkafka/admin/describe_configs_report.rb

@@ -6,7 +6,10 @@ module Rdkafka
     class DescribeConfigsReport
       attr_reader :resources
 
-      # @param config_entries [FFI::Pointer] pointer to config entries array
+      # @param config_entries [FFI::Pointer] pointer to the event-owned config entries array.
+      #   It is only read here - the array is destroyed together with the result event by the
+      #   background event callback, so this report must be built while the event is alive and
+      #   must copy everything it needs into Ruby objects.
       # @param entry_count [Integer] number of config entries
       def initialize(config_entries:, entry_count:)
         @resources = []
@@ -19,7 +22,7 @@ module Rdkafka
           .each do |config_resource_result_ptr|
             config_resource_result = ConfigResourceBindingResult.new(config_resource_result_ptr)
 
-            pointer_to_size_t = FFI::MemoryPointer.new(:int32)
+            pointer_to_size_t = FFI::MemoryPointer.new(:size_t)
             configs_ptr = Bindings.rd_kafka_ConfigResource_configs(
               config_resource_result_ptr,
               pointer_to_size_t
@@ -32,10 +35,6 @@ module Rdkafka
 
             @resources << config_resource_result
           end
-      ensure
-        return if config_entries == FFI::Pointer::NULL
-
-        Bindings.rd_kafka_ConfigResource_destroy_array(config_entries, entry_count)
       end
 
       private

data/lib/rdkafka/admin/incremental_alter_configs_handle.rb

@@ -5,30 +5,19 @@ module Rdkafka
     # Handle for incremental alter configs operation
     class IncrementalAlterConfigsHandle < AbstractHandle
       layout :pending, :bool,
-        :response, :int,
-        :response_string, :pointer,
-        :config_entries, :pointer,
-        :entry_count, :int
+        :response, :int
 
       # @return [String] the name of the operation.
       def operation_name
         "incremental alter configs"
       end
 
-      # @return [DescribeAclReport] instance with an array of acls that matches the request filters.
-      def create_result
-        IncrementalAlterConfigsReport.new(
-          config_entries: self[:config_entries],
-          entry_count: self[:entry_count]
-        )
-      end
-
       # Raises an error if the operation failed
       # @raise [RdkafkaError]
       def raise_error
         raise RdkafkaError.new(
           self[:response],
-          broker_message: self[:response_string].read_string
+          broker_message: broker_message
         )
       end
     end

data/lib/rdkafka/admin/incremental_alter_configs_report.rb

@@ -6,7 +6,10 @@ module Rdkafka
     class IncrementalAlterConfigsReport
       attr_reader :resources
 
-      # @param config_entries [FFI::Pointer] pointer to config entries array
+      # @param config_entries [FFI::Pointer] pointer to the event-owned config entries array.
+      #   It is only read here - the array is destroyed together with the result event by the
+      #   background event callback, so this report must be built while the event is alive and
+      #   must copy everything it needs into Ruby objects.
       # @param entry_count [Integer] number of config entries
       def initialize(config_entries:, entry_count:)
         @resources = []
@@ -19,7 +22,7 @@ module Rdkafka
           .each do |config_resource_result_ptr|
             config_resource_result = ConfigResourceBindingResult.new(config_resource_result_ptr)
 
-            pointer_to_size_t = FFI::MemoryPointer.new(:int32)
+            pointer_to_size_t = FFI::MemoryPointer.new(:size_t)
             configs_ptr = Bindings.rd_kafka_ConfigResource_configs(
               config_resource_result_ptr,
               pointer_to_size_t
@@ -32,10 +35,6 @@ module Rdkafka
 
             @resources << config_resource_result
           end
-      ensure
-        return if config_entries == FFI::Pointer::NULL
-
-        Bindings.rd_kafka_ConfigResource_destroy_array(config_entries, entry_count)
       end
 
       private

data/lib/rdkafka/admin/list_offsets_handle.rb

@@ -5,30 +5,19 @@ module Rdkafka
     # Handle for list offsets operation
     class ListOffsetsHandle < AbstractHandle
       layout :pending, :bool,
-        :response, :int,
-        :response_string, :pointer,
-        :result_infos, :pointer,
-        :result_count, :int
+        :response, :int
 
       # @return [String] the name of the operation.
       def operation_name
         "list offsets"
       end
 
-      # @return [ListOffsetsReport] instance with partition offset information.
-      def create_result
-        ListOffsetsReport.new(
-          result_infos: self[:result_infos],
-          result_count: self[:result_count]
-        )
-      end
-
       # Raises an error if the operation failed
       # @raise [RdkafkaError]
       def raise_error
         raise RdkafkaError.new(
           self[:response],
-          broker_message: self[:response_string].read_string
+          broker_message: broker_message
         )
       end
     end

data/lib/rdkafka/admin/list_offsets_report.rb

@@ -36,7 +36,7 @@ module Rdkafka
 
       # Validates the partition result and raises an error if invalid
       # @param result_info_ptr [FFI::Pointer] pointer to the result info
-      # @raise [RdkafkaError] when the partition has an error
+      # @raise [RdkafkaError] when the partition has an error, naming the failing partition
       def validate!(result_info_ptr)
         tp_ptr = Bindings.rd_kafka_ListOffsetsResultInfo_topic_partition(result_info_ptr)
         tp = Bindings::TopicPartition.new(tp_ptr)
@@ -44,7 +44,10 @@ module Rdkafka
 
         return if code.zero?
 
-        raise RdkafkaError.new(code)
+        raise RdkafkaError.new(
+          code,
+          "Error querying offsets for partition #{tp[:partition]} of '#{tp[:topic]}'"
+        )
       end
     end
   end