Addressing deprecation warnings

Signed-off-by: Soby Chacko <[email protected]>

Author: sobychacko

spring-kafka/src/main/java/org/springframework/kafka/aot/KafkaRuntimeHints.java

@@ -74,7 +74,6 @@
  */
 public class KafkaRuntimeHints implements RuntimeHintsRegistrar {
 
-	@SuppressWarnings("deprecation")
 	@Override
 	public void registerHints(RuntimeHints hints, @Nullable ClassLoader classLoader) {
 		ReflectionHints reflectionHints = hints.reflection();
@@ -106,8 +105,7 @@ public void registerHints(RuntimeHints hints, @Nullable ClassLoader classLoader)
 					KafkaListenerAnnotationBeanPostProcessor.class)
 				.forEach(type -> reflectionHints.registerType(type,
 						builder -> builder.withMembers(MemberCategory.INVOKE_DECLARED_CONSTRUCTORS,
-								MemberCategory.INVOKE_DECLARED_METHODS,
-								MemberCategory.INTROSPECT_PUBLIC_METHODS)));
+								MemberCategory.INVOKE_DECLARED_METHODS)));
 
 		Stream.of(
 					KafkaBootstrapConfiguration.class,

spring-kafka/src/main/java/org/springframework/kafka/event/NonResponsiveConsumerEvent.java

@@ -41,7 +41,7 @@ public class NonResponsiveConsumerEvent extends KafkaEvent {
 
 	private final String listenerId;
 
-	private final @Nullable List<TopicPartition> topicPartitions;
+	private final transient @Nullable List<TopicPartition> topicPartitions;
 
 	private transient final Consumer<?, ?> consumer;
 

spring-kafka/src/main/java/org/springframework/kafka/streams/RecoveringDeserializationExceptionHandler.java

@@ -23,7 +23,7 @@
 import org.apache.commons.logging.LogFactory;
 import org.apache.kafka.clients.consumer.ConsumerRecord;
 import org.apache.kafka.streams.errors.DeserializationExceptionHandler;
-import org.apache.kafka.streams.processor.ProcessorContext;
+import org.apache.kafka.streams.errors.ErrorHandlerContext;
 import org.jspecify.annotations.Nullable;
 
 import org.springframework.kafka.listener.ConsumerRecordRecoverer;
@@ -56,7 +56,7 @@ public RecoveringDeserializationExceptionHandler(ConsumerRecordRecoverer recover
 	}
 
 	@Override
-	public DeserializationHandlerResponse handle(ProcessorContext context, ConsumerRecord<byte[], byte[]> record,
+	public DeserializationHandlerResponse handle(ErrorHandlerContext context, ConsumerRecord<byte[], byte[]> record,
 			Exception exception) {
 
 		if (this.recoverer == null) {

spring-kafka/src/test/java/org/springframework/kafka/listener/MockConsumerTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2023-2024 the original author or authors.
+ * Copyright 2023-2025 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -28,7 +28,6 @@
 
 import org.apache.kafka.clients.consumer.ConsumerRecord;
 import org.apache.kafka.clients.consumer.MockConsumer;
-import org.apache.kafka.clients.consumer.OffsetResetStrategy;
 import org.apache.kafka.common.TopicPartition;
 import org.apache.kafka.common.header.internals.RecordHeaders;
 import org.apache.kafka.common.record.TimestampType;
@@ -87,7 +86,7 @@ public void foo(String in) {
 
 		@Bean
 		public ConsumerFactory<String, String> consumerFactory() {
-			MockConsumer<String, String> consumer = new MockConsumer<>(OffsetResetStrategy.EARLIEST);
+			MockConsumer<String, String> consumer = new MockConsumer<>("earliest");
 			TopicPartition topicPartition0 = new TopicPartition("foo", 0);
 			TopicPartition topicPartition1 = new TopicPartition("foo", 1);
 			TopicPartition topicPartition2 = new TopicPartition("foo", 2);

spring-kafka/src/test/java/org/springframework/kafka/streams/RecoveringDeserializationExceptionHandlerTests.java

@@ -34,8 +34,8 @@
 import org.apache.kafka.streams.StreamsBuilder;
 import org.apache.kafka.streams.StreamsConfig;
 import org.apache.kafka.streams.errors.DeserializationExceptionHandler.DeserializationHandlerResponse;
+import org.apache.kafka.streams.errors.ErrorHandlerContext;
 import org.apache.kafka.streams.kstream.KStream;
-import org.apache.kafka.streams.processor.ProcessorContext;
 import org.apache.kafka.streams.processor.WallclockTimestampExtractor;
 import org.junit.jupiter.api.Test;
 
@@ -94,9 +94,9 @@ void viaStringProperty() {
 				Recoverer.class.getName());
 		handler.configure(configs);
 		assertThat(KafkaTestUtils.getPropertyValue(handler, "recoverer")).isInstanceOf(Recoverer.class);
-		assertThat(handler.handle((ProcessorContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
+		assertThat(handler.handle((ErrorHandlerContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
 				new IllegalArgumentException())).isEqualTo(DeserializationHandlerResponse.CONTINUE);
-		assertThat(handler.handle((ProcessorContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
+		assertThat(handler.handle((ErrorHandlerContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
 				new IllegalStateException())).isEqualTo(DeserializationHandlerResponse.FAIL);
 	}
 
@@ -107,9 +107,9 @@ void viaClassProperty() {
 		configs.put(RecoveringDeserializationExceptionHandler.KSTREAM_DESERIALIZATION_RECOVERER, Recoverer.class);
 		handler.configure(configs);
 		assertThat(KafkaTestUtils.getPropertyValue(handler, "recoverer")).isInstanceOf(Recoverer.class);
-		assertThat(handler.handle((ProcessorContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
+		assertThat(handler.handle((ErrorHandlerContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
 				new IllegalArgumentException())).isEqualTo(DeserializationHandlerResponse.CONTINUE);
-		assertThat(handler.handle((ProcessorContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
+		assertThat(handler.handle((ErrorHandlerContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
 				new IllegalStateException())).isEqualTo(DeserializationHandlerResponse.FAIL);
 	}
 
@@ -121,16 +121,16 @@ void viaObjectProperty() {
 		configs.put(RecoveringDeserializationExceptionHandler.KSTREAM_DESERIALIZATION_RECOVERER, rec);
 		handler.configure(configs);
 		assertThat(KafkaTestUtils.getPropertyValue(handler, "recoverer")).isSameAs(rec);
-		assertThat(handler.handle((ProcessorContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
+		assertThat(handler.handle((ErrorHandlerContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
 				new IllegalArgumentException())).isEqualTo(DeserializationHandlerResponse.CONTINUE);
-		assertThat(handler.handle((ProcessorContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
+		assertThat(handler.handle((ErrorHandlerContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
 				new IllegalStateException())).isEqualTo(DeserializationHandlerResponse.FAIL);
 	}
 
 	@Test
 	void withNoRecoverer() {
 		RecoveringDeserializationExceptionHandler handler = new RecoveringDeserializationExceptionHandler();
-		assertThat(handler.handle((ProcessorContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
+		assertThat(handler.handle((ErrorHandlerContext) null, new ConsumerRecord<>("foo", 0, 0, null, null),
 				new IllegalArgumentException())).isEqualTo(DeserializationHandlerResponse.FAIL);
 	}