karafka 2.6.0.beta1 → 2.6.0.rc1

data/lib/karafka/pro/processing/consumer_groups/jobs/eofed_non_blocking.rb

@@ -31,9 +31,7 @@
 module Karafka
   module Pro
     module Processing
-      # Pro consumer-group-specific processing components
       module ConsumerGroups
-        # Pro jobs
         module Jobs
           # Non-Blocking version of the Eofed job
           # We use this version for LRJ topics for cases where saturated resources would not allow

data/lib/karafka/pro/processing/consumer_groups/jobs/periodic.rb

@@ -31,7 +31,6 @@
 module Karafka
   module Pro
     module Processing
-      # Pro consumer-group-specific processing components
       module ConsumerGroups
         # Pro jobs
         module Jobs
@@ -40,8 +39,8 @@ module Karafka
           class Periodic < Karafka::Processing::Jobs::Base
             self.action = :tick
 
-            # @param executor [Karafka::Pro::Processing::ConsumerGroups::Executor] pro executor that is suppose to
-            #   run a given job
+            # @param executor [Karafka::Pro::Processing::ConsumerGroups::Executor] pro executor that
+            #   is suppose to run a given job
             def initialize(executor)
               @executor = executor
               super()

data/lib/karafka/pro/processing/consumer_groups/jobs/periodic_non_blocking.rb

@@ -31,9 +31,7 @@
 module Karafka
   module Pro
     module Processing
-      # Pro consumer-group-specific processing components
       module ConsumerGroups
-        # Pro jobs
         module Jobs
           # Non-Blocking version of the Periodic job
           # We use this version for LRJ topics for cases where saturated resources would not allow

data/lib/karafka/pro/processing/consumer_groups/jobs/revoked_non_blocking.rb

@@ -30,11 +30,8 @@
 
 module Karafka
   module Pro
-    # Pro components related to processing part of Karafka
     module Processing
-      # Pro consumer-group-specific processing components
       module ConsumerGroups
-        # Pro jobs
         module Jobs
           # The revoked job type in a non-blocking variant.
           # This variant works "like" the regular revoked but does not block the queue.

data/lib/karafka/pro/processing/consumer_groups/jobs_builder.rb

@@ -42,8 +42,8 @@ module Karafka
           # @param executor [Karafka::Pro::Processing::ConsumerGroups::Executor]
           # @param messages [Karafka::Messages::Messages] messages batch to be consumed
           # @return [Karafka::Processing::ConsumerGroups::Jobs::Consume] blocking job
-          # @return [Karafka::Pro::Processing::ConsumerGroups::Jobs::ConsumeNonBlocking] non blocking
-          #   for lrj
+          # @return [Karafka::Pro::Processing::ConsumerGroups::Jobs::ConsumeNonBlocking] non
+          #   blocking for lrj
           def consume(executor, messages)
             if executor.topic.long_running_job?
               Jobs::ConsumeNonBlocking.new(executor, messages)
@@ -54,8 +54,9 @@ module Karafka
 
           # @param executor [Karafka::Pro::Processing::ConsumerGroups::Executor]
           # @return [Karafka::Processing::ConsumerGroups::Jobs::Eofed] eofed job for non LRJ
-          # @return [Karafka::Pro::Processing::ConsumerGroups::Jobs::EofedNonBlocking] eofed job that
-          #   is non-blocking, so when revocation job is scheduled for LRJ it also will not block
+          # @return [Karafka::Pro::Processing::ConsumerGroups::Jobs::EofedNonBlocking] eofed job
+          #   that is non-blocking, so when revocation job is scheduled for LRJ it also will not
+          #   block
           def eofed(executor)
             if executor.topic.long_running_job?
               Jobs::EofedNonBlocking.new(executor)

data/lib/karafka/pro/processing/consumer_groups/offset_metadata/consumer.rb

@@ -31,16 +31,12 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
-        # Offset Metadata support on the processing side
         module OffsetMetadata
           # Extra API methods for offset metadata fetching
-          # @note Part of this feature API is embedded directly into the strategies because it alters
-          #   how marking methods (`#mark_as_consumed` and `#mark_as_consumed!`) operate. Because
-          #   of that, they had to be embedded into the strategies.
+          # @note Part of this feature API is embedded directly into the strategies because it
+          #   alters how marking methods (`#mark_as_consumed` and `#mark_as_consumed!`) operate.
+          #   Because of that, they had to be embedded into the strategies.
           module Consumer
             # @param cache [Boolean] should we use cached result if present (true by default)
             # @return [false, Object] false in case we do not own the partition anymore or

data/lib/karafka/pro/processing/consumer_groups/offset_metadata/fetcher.rb

@@ -31,19 +31,16 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         # Offset Metadata support on the processing side
         module OffsetMetadata
           # This fetcher is responsible for fetching and caching committed offsets metadata
           # information.
           #
-          # By design we fetch all information for a requested topic assignments. Not all topics from
-          # the same subscription group may need metadata and even if, we can run the few smaller
-          # queries. This approach prevents us from querying all assigned topics data in one go
-          # preventing excessive queries.
+          # By design we fetch all information for a requested topic assignments. Not all topics
+          # from the same subscription group may need metadata and even if, we can run the few
+          # smaller queries. This approach prevents us from querying all assigned topics data in one
+          # go preventing excessive queries.
           #
           # Since the assumption is, that user will not have to reach out for the later metadata
           # since it is produced in the context of a given consumer assignment, we can cache the
@@ -64,7 +61,8 @@ module Karafka
               @tpls = {}
             end
 
-            # Registers a client of a given subscription group, so we can use it for queries later on
+            # Registers a client of a given subscription group, so we can use it for queries later
+            # on
             # @param client [Karafka::Connection::Client]
             # @note Since we store the client reference and not the underlying rdkafka consumer
             #   instance, we do not have to deal with the recovery as it is abstracted away
@@ -104,8 +102,8 @@ module Karafka
 
             # Clears cache of a given subscription group. It is triggered on assignment changes.
             #
-            # @param subscription_group [Karafka::Routing::SubscriptionGroup] subscription group that
-            #   we want to clear.
+            # @param subscription_group [Karafka::Routing::SubscriptionGroup] subscription group
+            #   that we want to clear.
             def clear(subscription_group)
               @mutexes.fetch(subscription_group).synchronize do
                 @tpls[subscription_group].clear

data/lib/karafka/pro/processing/consumer_groups/offset_metadata/listener.rb

@@ -31,9 +31,6 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module OffsetMetadata
           # Keeps track of rebalances and updates the fetcher

data/lib/karafka/pro/processing/consumer_groups/parallel_segments/filters/base.rb

@@ -31,15 +31,13 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module ParallelSegments
           # Module for filters injected into the processing pipeline of each of the topics used
           # within the parallel segmented consumer groups
           module Filters
-            # Base class for filters for parallel segments that deal with different feature scenarios
+            # Base class for filters for parallel segments that deal with different feature
+            # scenarios
             class Base < Pro::Processing::ConsumerGroups::Filters::Base
               # @param segment_id [Integer] numeric id of the parallel segment group to use with the
               #   partitioner and reducer for segment matching comparison
@@ -82,7 +80,7 @@ module Karafka
 
                 @reducer.call(message_segment_key)
               rescue => e
-                # @see `#partition` method error handling doc
+                # See `#partition` method error handling doc
                 Karafka.monitor.instrument(
                   "error.occurred",
                   caller: self,

data/lib/karafka/pro/processing/consumer_groups/parallel_segments/filters/default.rb

@@ -31,9 +31,6 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         # Processing components namespace for parallel segments feature
         module ParallelSegments

data/lib/karafka/pro/processing/consumer_groups/parallel_segments/filters/mom.rb

@@ -31,9 +31,6 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module ParallelSegments
           module Filters

data/lib/karafka/pro/processing/consumer_groups/partitioner.rb

@@ -31,16 +31,13 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         # Pro partitioner that can distribute work based on the virtual partitioner settings
         class Partitioner < Karafka::Processing::ConsumerGroups::Partitioner
           # @param topic [String] topic name
           # @param messages [Array<Karafka::Messages::Message>] karafka messages
-          # @param coordinator [Karafka::Pro::Processing::ConsumerGroups::Coordinator] processing coordinator that
-          #   will be used with those messages
+          # @param coordinator [Karafka::Pro::Processing::ConsumerGroups::Coordinator]
+          #   processing coordinator that will be used with those messages
           # @yieldparam [Integer] group id
           # @yieldparam [Array<Karafka::Messages::Message>] karafka messages
           def call(topic, messages, coordinator)

data/lib/karafka/pro/processing/consumer_groups/periodic_job/consumer.rb

@@ -31,9 +31,6 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         # Namespace for periodic jobs related processing APIs
         module PeriodicJob
@@ -64,8 +61,8 @@ module Karafka
             end
 
             # Runs the on-schedule tick periodic operations
-            # This method is an alias but is part of the naming convention used for other flows, this
-            # is why we do not reference the `handle_before_schedule_tick` directly
+            # This method is an alias but is part of the naming convention used for other flows,
+            # this is why we do not reference the `handle_before_schedule_tick` directly
             def on_before_schedule_tick
               handle_before_schedule_tick
             end

data/lib/karafka/pro/processing/consumer_groups/piping/consumer.rb

@@ -31,9 +31,6 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         # All code needed for messages piping in Karafka
         module Piping
@@ -48,8 +45,8 @@ module Karafka
             private_constant :EMPTY_HASH
 
             # Pipes given message to the provided topic with expected details. Useful for
-            # pass-through operations where deserialization is not needed. Upon usage it will include
-            # all the source headers + meta headers about the source of message.
+            # pass-through operations where deserialization is not needed. Upon usage it will
+            # include all the source headers + meta headers about the source of message.
             #
             # @param topic [String, Symbol] where we want to send the message
             # @param message [Karafka::Messages::Message] source message to pipe

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/dlq_ftr_lrj_mom.rb

@@ -31,17 +31,14 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj
-            # ActiveJob enabled
-            # DLQ enabled
-            # Filtering enabled
-            # Long-Running Job enabled
-            # Manual offset management enabled
+            # - ActiveJob enabled
+            # - DLQ enabled
+            # - Filtering enabled
+            # - Long-Running Job enabled
+            # - Manual offset management enabled
             module DlqFtrLrjMom
               include Strategies::Aj::FtrMom
               include Strategies::Aj::DlqMom
@@ -67,7 +64,7 @@ module Karafka
 
                       # :seek and :pause are fully handled by handle_post_filtering
                       # For :skip we still need to resume the LRJ MAX_PAUSE_TIME pause
-                      return unless coordinator.filter.action == :skip
+                      return unless coordinator.filter.skip?
                     elsif !revoked?
                       # no need to check for manual seek because AJ consumer is internal and
                       # fully controlled by us
@@ -77,6 +74,10 @@ module Karafka
                     resume
                   else
                     apply_dlq_flow do
+                      # LRJ revocation runs concurrently with the job. If the partition was
+                      # revoked we must not dispatch: the new owner will reprocess and dispatch
+                      return resume if revoked?
+
                       skippable_message, = find_skippable_message
                       dispatch_to_dlq(skippable_message) if dispatch_to_dlq?
                       mark_dispatched_to_dlq(skippable_message)

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/dlq_ftr_lrj_mom_vp.rb

@@ -31,18 +31,15 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj
-            # ActiveJob enabled
-            # DLQ enabled
-            # Filtering enabled
-            # Long-Running Job enabled
-            # Manual offset management enabled
-            # Virtual Partitions enabled
+            # - ActiveJob enabled
+            # - DLQ enabled
+            # - Filtering enabled
+            # - Long-Running Job enabled
+            # - Manual offset management enabled
+            # - Virtual Partitions enabled
             module DlqFtrLrjMomVp
               include Strategies::Aj::FtrMom
               include Strategies::Aj::DlqMomVp
@@ -73,7 +70,7 @@ module Karafka
 
                       # :seek and :pause are fully handled by handle_post_filtering
                       # For :skip we still need to resume the LRJ MAX_PAUSE_TIME pause
-                      return unless coordinator.filter.action == :skip
+                      return unless coordinator.filter.skip?
                     elsif !revoked?
                       # no need to check for manual seek because AJ consumer is internal and
                       # fully controlled by us
@@ -83,6 +80,10 @@ module Karafka
                     resume
                   else
                     apply_dlq_flow do
+                      # LRJ revocation runs concurrently with the job. If the partition was
+                      # revoked we must not dispatch: the new owner will reprocess and dispatch
+                      return resume if revoked?
+
                       skippable_message, = find_skippable_message
                       dispatch_to_dlq(skippable_message) if dispatch_to_dlq?
                       mark_dispatched_to_dlq(skippable_message)

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/dlq_ftr_mom.rb

@@ -31,9 +31,6 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/dlq_ftr_mom_vp.rb

@@ -31,9 +31,6 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/dlq_lrj_mom.rb

@@ -31,16 +31,13 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj
-            # ActiveJob enabled
-            # DLQ enabled
-            # Long-Running Job enabled
-            # Manual offset management enabled
+            # - ActiveJob enabled
+            # - DLQ enabled
+            # - Long-Running Job enabled
+            # - Manual offset management enabled
             #
             # This case is a bit of special. Please see the `AjDlqMom` for explanation on how the
             # offset management works in this case.
@@ -70,6 +67,10 @@ module Karafka
                     resume
                   else
                     apply_dlq_flow do
+                      # LRJ revocation runs concurrently with the job. If the partition was
+                      # revoked we must not dispatch: the new owner will reprocess and dispatch
+                      return resume if revoked?
+
                       skippable_message, = find_skippable_message
                       dispatch_to_dlq(skippable_message) if dispatch_to_dlq?
                       mark_dispatched_to_dlq(skippable_message)

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/dlq_lrj_mom_vp.rb

@@ -31,17 +31,14 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj
-            # ActiveJob enabled
-            # DLQ enabled
-            # Long-Running Job enabled
-            # Manual offset management enabled
-            # Virtual Partitions enabled
+            # - ActiveJob enabled
+            # - DLQ enabled
+            # - Long-Running Job enabled
+            # - Manual offset management enabled
+            # - Virtual Partitions enabled
             #
             # This case is a bit of special. Please see the `Aj::DlqMom` for explanation on how the
             # offset management works in this case.
@@ -74,6 +71,10 @@ module Karafka
                     resume
                   else
                     apply_dlq_flow do
+                      # LRJ revocation runs concurrently with the job. If the partition was
+                      # revoked we must not dispatch: the new owner will reprocess and dispatch
+                      return resume if revoked?
+
                       skippable_message, = find_skippable_message
                       dispatch_to_dlq(skippable_message) if dispatch_to_dlq?
                       mark_dispatched_to_dlq(skippable_message)

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/dlq_mom.rb

@@ -31,18 +31,15 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj
-            # ActiveJob enabled
-            # DLQ enabled
-            # Manual offset management enabled
+            # - ActiveJob enabled
+            # - DLQ enabled
+            # - Manual offset management enabled
             #
-            # AJ has manual offset management on by default and the offset management is delegated to
-            # the AJ consumer. This means, we cannot mark as consumed always. We can only mark as
+            # AJ has manual offset management on by default and the offset management is delegated
+            # to the AJ consumer. This means, we cannot mark as consumed always. We can only mark as
             # consumed when we skip given job upon errors. In all the other scenarios marking as
             # consumed needs to happen in the AJ consumer on a per job basis.
             module DlqMom

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/dlq_mom_vp.rb

@@ -31,15 +31,12 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj
-            # ActiveJob enabled
-            # Manual offset management enabled
-            # Virtual Partitions enabled
+            # - ActiveJob enabled
+            # - Manual offset management enabled
+            # - Virtual Partitions enabled
             module DlqMomVp
               include Strategies::Default
               include Strategies::Dlq::Vp

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/ftr_lrj_mom.rb

@@ -31,16 +31,13 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj
-            # ActiveJob enabled
-            # Filtering enabled
-            # Long-Running Job enabled
-            # Manual offset management enabled
+            # - ActiveJob enabled
+            # - Filtering enabled
+            # - Long-Running Job enabled
+            # - Manual offset management enabled
             module FtrLrjMom
               include Strategies::Lrj::FtrMom
 

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/ftr_lrj_mom_vp.rb

@@ -31,17 +31,14 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj
-            # ActiveJob enabled
-            # Filtering enabled
-            # Long-Running Job enabled
-            # Manual offset management enabled
-            # Virtual Partitions enabled
+            # - ActiveJob enabled
+            # - Filtering enabled
+            # - Long-Running Job enabled
+            # - Manual offset management enabled
+            # - Virtual Partitions enabled
             module FtrLrjMomVp
               include Strategies::Vp::Default
               include Strategies::Lrj::FtrMom
@@ -69,7 +66,7 @@ module Karafka
 
                       # :seek and :pause are fully handled by handle_post_filtering
                       # For :skip we still need to resume the LRJ MAX_PAUSE_TIME pause
-                      return unless coordinator.filter.action == :skip
+                      return unless coordinator.filter.skip?
                     elsif !revoked?
                       # no need to check for manual seek because AJ consumer is internal and
                       # fully controlled by us

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/ftr_mom.rb

@@ -31,16 +31,12 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
-          # Namespace for ActiveJob related strategies
           module Aj
-            # ActiveJob enabled
-            # Filtering enabled
-            # Manual Offset management enabled
+            # - ActiveJob enabled
+            # - Filtering enabled
+            # - Manual Offset management enabled
             module FtrMom
               # Same as standard Mom::Ftr
               include Strategies::Mom::Ftr

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/ftr_mom_vp.rb

@@ -31,17 +31,13 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
-          # Namespace for ActiveJob related strategies
           module Aj
-            # ActiveJob enabled
-            # Filtering enabled
-            # Manual Offset management enabled
-            # Virtual partitions enabled
+            # - ActiveJob enabled
+            # - Filtering enabled
+            # - Manual Offset management enabled
+            # - Virtual partitions enabled
             module FtrMomVp
               include Strategies::Aj::FtrMom
               include Strategies::Aj::MomVp

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/lrj_mom.rb

@@ -31,15 +31,12 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj
-            # ActiveJob enabled
-            # Long-Running Job enabled
-            # Manual offset management enabled
+            # - ActiveJob enabled
+            # - Long-Running Job enabled
+            # - Manual offset management enabled
             module LrjMom
               # Same behaviour as Lrj::Mom
               include Strategies::Lrj::Mom

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/lrj_mom_vp.rb

@@ -31,16 +31,13 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           module Aj
-            # ActiveJob enabled
-            # Long-Running Job enabled
-            # Manual offset management enabled
-            # Virtual Partitions enabled
+            # - ActiveJob enabled
+            # - Long-Running Job enabled
+            # - Manual offset management enabled
+            # - Virtual Partitions enabled
             module LrjMomVp
               include Strategies::Default
               include Strategies::Vp::Default

data/lib/karafka/pro/processing/consumer_groups/strategies/aj/mom.rb

@@ -31,15 +31,12 @@
 module Karafka
   module Pro
     module Processing
-      # Consumer-group-specific Pro processing components (driven by rebalance callbacks and
-      # partition ticks). Parallel `ShareGroups` will live next to this namespace once KIP-932
-      # lands.
       module ConsumerGroups
         module Strategies
           # Namespace for ActiveJob related strategies
           module Aj
-            # ActiveJob enabled
-            # Manual Offset management enabled
+            # - ActiveJob enabled
+            # - Manual Offset management enabled
             module Mom
               # Standard ActiveJob strategy is the same one we use for Mom
               include Strategies::Mom::Default