`HealthIndicator` mit Details zu Rebalances des Consumers implementiert consumer/spring-consumer--health-indicator consumer/spring-consumer--health-indicator--2025-04-signal-spickzettel consumer/spring-consumer--health-indicator--2025-05-lvm consumer/spring-consumer--health-indicator--2025-05-lvm--spickzettel consumer/spring-consumer--health-indicator--2025-05-signal-spickzettel consumer/spring-consumer--health-indicator--2025-07-05
authorKai Moritz <kai@juplo.de>
Mon, 18 Nov 2024 06:38:42 +0000 (07:38 +0100)
committerKai Moritz <kai@juplo.de>
Wed, 16 Apr 2025 19:54:47 +0000 (21:54 +0200)
* `ConsumerHealthIndicator` refaktorisiert (DRY)
* `ConsumerHealthIndicator` refaktorisiert (Aufgaben klarer getrennt)
* `ConsumerHealthIndicator` refaktorisiert (Klareres Benennungs-Schema)
* `ConsumerHealthIndicator` refaktorisiert (Methoden aufgerÀumt)

README.sh
build.gradle
docker/docker-compose.yml
pom.xml
src/main/java/de/juplo/kafka/ApplicationConfiguration.java
src/main/java/de/juplo/kafka/ConsumerHealthIndicator.java [new file with mode: 0644]
src/main/java/de/juplo/kafka/ConsumerHealthIndicatorAwareRebalanceListener.java [new file with mode: 0644]
src/main/java/de/juplo/kafka/ExampleConsumer.java
src/main/resources/application.yml

index 203c22b..90e25d5 100755 (executable)
--- a/README.sh
+++ b/README.sh
@@ -1,6 +1,6 @@
 #!/bin/bash
 
-IMAGE=juplo/spring-consumer:1.1-record-handler-SNAPSHOT
+IMAGE=juplo/spring-consumer:1.1-health-indicator-SNAPSHOT
 
 if [ "$1" = "cleanup" ]
 then
index f700918..12642ad 100644 (file)
@@ -8,7 +8,7 @@ plugins {
 }
 
 group = 'de.juplo.kafka'
-version = '1.1-record-handler-SNAPSHOT'
+version = '1.1-health-indicator-SNAPSHOT'
 
 java {
        toolchain {
index bf306f6..adf2acd 100644 (file)
@@ -140,7 +140,7 @@ services:
     command: kafka:9092 test producer
 
   consumer:
-    image: juplo/spring-consumer:1.1-record-handler-SNAPSHOT
+    image: juplo/spring-consumer:1.1-health-indicator-SNAPSHOT
     environment:
       spring.kafka.bootstrap-servers: kafka:9092
       spring.kafka.client-id: consumer
@@ -149,7 +149,7 @@ services:
       juplo.consumer.topic: test
 
   peter:
-    image: juplo/spring-consumer:1.1-record-handler-SNAPSHOT
+    image: juplo/spring-consumer:1.1-health-indicator-SNAPSHOT
     environment:
       spring.kafka.bootstrap-servers: kafka:9092
       spring.kafka.client-id: consumer
@@ -158,7 +158,7 @@ services:
       juplo.consumer.topic: test
 
   ute:
-    image: juplo/spring-consumer:1.1-record-handler-SNAPSHOT
+    image: juplo/spring-consumer:1.1-health-indicator-SNAPSHOT
     environment:
       spring.kafka.bootstrap-servers: kafka:9092
       spring.kafka.client-id: consumer
diff --git a/pom.xml b/pom.xml
index 7590d25..074909e 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -15,7 +15,7 @@
   <artifactId>spring-consumer</artifactId>
   <name>Spring Consumer</name>
   <description>Super Simple Consumer-Group, that is implemented as Spring-Boot application and configured by Spring Kafka</description>
-  <version>1.1-record-handler-SNAPSHOT</version>
+  <version>1.1-health-indicator-SNAPSHOT</version>
 
   <properties>
     <java.version>21</java.version>
index 87331b3..cf7fefc 100644 (file)
@@ -9,6 +9,8 @@ import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 import org.springframework.kafka.core.ConsumerFactory;
 
+import java.time.Clock;
+
 
 @Configuration
 @EnableConfigurationProperties(ApplicationProperties.class)
@@ -19,6 +21,7 @@ public class ApplicationConfiguration
   public ExampleConsumer<String, String> exampleConsumer(
     Consumer<String, String> kafkaConsumer,
     RecordHandler<String, String> recordHandler,
+    ConsumerHealthIndicatorAwareRebalanceListener consumerHealthIndicatorAwareRebalanceListener,
     ApplicationProperties properties,
     KafkaProperties kafkaProperties,
     ConfigurableApplicationContext applicationContext)
@@ -29,6 +32,7 @@ public class ApplicationConfiguration
         properties.getConsumerProperties().getTopic(),
         kafkaConsumer,
         recordHandler,
+        consumerHealthIndicatorAwareRebalanceListener,
         () -> applicationContext.close());
   }
 
@@ -38,6 +42,24 @@ public class ApplicationConfiguration
     return (topic, partition, offset, key, value) -> log.info("No-Ops Handler called for {}={}", key, value);
   }
 
+  @Bean
+  public ConsumerHealthIndicatorAwareRebalanceListener rebalanceListener(ConsumerHealthIndicator consumerHealthIndicator)
+  {
+    return new ConsumerHealthIndicatorAwareRebalanceListener(consumerHealthIndicator);
+  }
+
+  @Bean
+  public ConsumerHealthIndicator consumerHealthIndicator(Clock clock)
+  {
+    return new ConsumerHealthIndicator(clock);
+  }
+
+  @Bean
+  public Clock clock()
+  {
+    return Clock.systemDefaultZone();
+  }
+
   @Bean(destroyMethod = "")
   public Consumer<?, ?> kafkaConsumer(ConsumerFactory<?, ?> consumerFactory)
   {
diff --git a/src/main/java/de/juplo/kafka/ConsumerHealthIndicator.java b/src/main/java/de/juplo/kafka/ConsumerHealthIndicator.java
new file mode 100644 (file)
index 0000000..30fb732
--- /dev/null
@@ -0,0 +1,116 @@
+package de.juplo.kafka;
+
+import org.apache.kafka.common.TopicPartition;
+import org.springframework.boot.actuate.health.Health;
+import org.springframework.boot.actuate.health.HealthIndicator;
+
+import java.time.Clock;
+import java.time.ZonedDateTime;
+import java.util.*;
+
+
+public class ConsumerHealthIndicator implements HealthIndicator
+{
+  private final Clock clock;
+
+  private volatile RebalancingState rebalancingState;
+  private volatile List<RecordedState> history;
+  private volatile List<Partition> assignedPartitions = List.of();
+
+
+  public ConsumerHealthIndicator(Clock clock)
+  {
+    this.clock = clock;
+
+    rebalancingState = RebalancingState.STARTING;
+    RecordedState sat = new RecordedState(ZonedDateTime.now(clock), rebalancingState, List.of());
+    history = List.of(sat);
+  }
+
+
+  @Override
+  public Health getHealth(boolean includeDetails)
+  {
+    Health.Builder healthBuilder = getHealthBuilder();
+
+    if (includeDetails)
+    {
+      healthBuilder.withDetail("rebalancing_state", rebalancingState);
+      healthBuilder.withDetail("history", history);
+    }
+
+    return healthBuilder.build();
+  }
+
+  @Override
+  public Health health()
+  {
+    return getHealthBuilder().build();
+  }
+
+  private Health.Builder getHealthBuilder()
+  {
+    return rebalancingState == RebalancingState.RUNNING
+      ? new Health.Builder().up()
+      : new Health.Builder().status(rebalancingState.name());
+  }
+
+
+  public void partitionsAssigned(Collection<TopicPartition> partitions)
+  {
+    List<Partition> newAssignedPartitions = new LinkedList<>(this.assignedPartitions);
+    partitions.forEach(tp -> newAssignedPartitions.add(new Partition(tp.topic(), tp.partition())));
+    Collections.sort(newAssignedPartitions, partitionComparator);
+
+    updateAndRecordState(RebalancingState.RUNNING, newAssignedPartitions);
+  }
+
+  public void partitionsRevoked(Collection<TopicPartition> partitions)
+  {
+    List<Partition> newAssignedPartitions = new LinkedList<>(this.assignedPartitions);
+    partitions.forEach(tp -> newAssignedPartitions.remove(new Partition(tp.topic(), tp.partition())));
+
+    updateAndRecordState(RebalancingState.REBALANCING, newAssignedPartitions);
+  }
+
+  public void partitionsLost(Collection<TopicPartition> partitions)
+  {
+    updateAndRecordState(RebalancingState.FENCED, List.of());
+  }
+
+  private void updateAndRecordState(
+    RebalancingState newRebalancingState,
+    List<Partition> newAssignedPartitions)
+  {
+    List<RecordedState> newHistory = new LinkedList<>();
+    newHistory.add(new RecordedState(
+      ZonedDateTime.now(clock),
+      newRebalancingState,
+      newAssignedPartitions));
+    newHistory.addAll(this.history);
+    if(newHistory.size() > 10)
+    {
+      newHistory.removeLast();
+    }
+    this.rebalancingState = newRebalancingState;
+    this.assignedPartitions = newAssignedPartitions;
+    this.history = newHistory;
+  }
+
+
+  enum RebalancingState { STARTING, FENCED, REBALANCING, RUNNING }
+
+  public record Partition(String topic, Integer partition) {}
+  public record RecordedState(
+    ZonedDateTime time,
+    RebalancingState rebalancingState,
+    List<Partition> assignedPartitions) {}
+
+  private final static Comparator<Partition> partitionComparator = (tp1, tp2) ->
+  {
+    int result = tp1.topic().compareTo(tp2.topic());
+    return result == 0
+      ? tp1.partition() - tp2.partition()
+      : result;
+  };
+}
diff --git a/src/main/java/de/juplo/kafka/ConsumerHealthIndicatorAwareRebalanceListener.java b/src/main/java/de/juplo/kafka/ConsumerHealthIndicatorAwareRebalanceListener.java
new file mode 100644 (file)
index 0000000..3339ac9
--- /dev/null
@@ -0,0 +1,33 @@
+package de.juplo.kafka;
+
+import lombok.RequiredArgsConstructor;
+import org.apache.kafka.clients.consumer.ConsumerRebalanceListener;
+import org.apache.kafka.common.TopicPartition;
+
+import java.util.Collection;
+
+
+@RequiredArgsConstructor
+public class ConsumerHealthIndicatorAwareRebalanceListener implements ConsumerRebalanceListener
+{
+  private final ConsumerHealthIndicator consumerHealthIndicator;
+
+
+  @Override
+  public void onPartitionsAssigned(Collection<TopicPartition> partitions)
+  {
+    consumerHealthIndicator.partitionsAssigned(partitions);
+  }
+
+  @Override
+  public void onPartitionsRevoked(Collection<TopicPartition> partitions)
+  {
+    consumerHealthIndicator.partitionsRevoked(partitions);
+  }
+
+  @Override
+  public void onPartitionsLost(Collection<TopicPartition> partitions)
+  {
+    consumerHealthIndicator.partitionsLost(partitions);
+  }
+}
index e5a8b3d..bd5cac7 100644 (file)
@@ -2,6 +2,7 @@ package de.juplo.kafka;
 
 import lombok.extern.slf4j.Slf4j;
 import org.apache.kafka.clients.consumer.Consumer;
+import org.apache.kafka.clients.consumer.ConsumerRebalanceListener;
 import org.apache.kafka.clients.consumer.ConsumerRecord;
 import org.apache.kafka.clients.consumer.ConsumerRecords;
 import org.apache.kafka.common.errors.WakeupException;
@@ -17,6 +18,7 @@ public class ExampleConsumer<K, V> implements Runnable
   private final String topic;
   private final Consumer<K, V> consumer;
   private final RecordHandler<K, V> recordHandler;
+  private final ConsumerRebalanceListener rebalanceListener;
   private final Thread workerThread;
   private final Runnable closeCallback;
 
@@ -28,12 +30,14 @@ public class ExampleConsumer<K, V> implements Runnable
     String topic,
     Consumer<K, V> consumer,
     RecordHandler<K, V> recordHandler,
+    ConsumerRebalanceListener rebalanceListener,
     Runnable closeCallback)
   {
     this.id = clientId;
     this.topic = topic;
     this.consumer = consumer;
     this.recordHandler = recordHandler;
+    this.rebalanceListener = rebalanceListener;
 
     workerThread = new Thread(this, "ExampleConsumer Worker-Thread");
     workerThread.start();
@@ -48,7 +52,7 @@ public class ExampleConsumer<K, V> implements Runnable
     try
     {
       log.info("{} - Subscribing to topic {}", id, topic);
-      consumer.subscribe(Arrays.asList(topic));
+      consumer.subscribe(Arrays.asList(topic), rebalanceListener);
 
       while (true)
       {
index 71dddda..8b37911 100644 (file)
@@ -3,6 +3,8 @@ juplo:
     topic: test
 management:
   endpoint:
+    health:
+      show-details: ALWAYS
     shutdown:
       enabled: true
   endpoints: