Skip to content

Commit db0f958

Browse files
committed
Upgrade kafka-clients to 3.2.0; other upgrades
Fix deprecation warnings.
1 parent 02d3b72 commit db0f958

File tree

5 files changed

+15
-13
lines changed

5 files changed

+15
-13
lines changed

build.gradle

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -64,17 +64,17 @@ ext {
6464
jaywayJsonPathVersion = '2.6.0'
6565
junit4Version = '4.13.2'
6666
junitJupiterVersion = '5.8.2'
67-
kafkaVersion = '3.1.0'
67+
kafkaVersion = '3.2.0'
6868
log4jVersion = '2.17.2'
6969
micrometerVersion = '1.10.0-SNAPSHOT'
7070
micrometerTracingVersion = '1.0.0-SNAPSHOT'
7171
mockitoVersion = '4.0.0'
72-
reactorVersion = '2020.0.17'
72+
reactorVersion = '2020.0.18'
7373
scalaVersion = '2.13'
74-
springBootVersion = '2.6.4' // docs module
75-
springDataVersion = '2022.0.0-M3'
76-
springRetryVersion = '1.3.2'
77-
springVersion = '6.0.0-M3'
74+
springBootVersion = '2.6.7' // docs module
75+
springDataVersion = '2022.0.0-M4'
76+
springRetryVersion = '1.3.3'
77+
springVersion = '6.0.0-M4'
7878
zookeeperVersion = '3.6.3'
7979

8080
idPrefix = 'kafka'

spring-kafka/src/main/java/org/springframework/kafka/annotation/KafkaListenerAnnotationBeanPostProcessor.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,6 @@
8787
import org.springframework.kafka.listener.KafkaListenerErrorHandler;
8888
import org.springframework.kafka.listener.adapter.RecordFilterStrategy;
8989
import org.springframework.kafka.retrytopic.RetryTopicBeanNames;
90-
import org.springframework.kafka.retrytopic.RetryTopicBootstrapper;
9190
import org.springframework.kafka.retrytopic.RetryTopicConfiguration;
9291
import org.springframework.kafka.retrytopic.RetryTopicConfigurer;
9392
import org.springframework.kafka.support.TopicPartitionOffset;
@@ -527,8 +526,9 @@ private void bootstrapRetryTopicIfNecessary() {
527526
BeanDefinitionRegistry registry = (BeanDefinitionRegistry) this.beanFactory;
528527
if (!registry.containsBeanDefinition("internalRetryTopicBootstrapper")) {
529528
registry.registerBeanDefinition("internalRetryTopicBootstrapper",
530-
new RootBeanDefinition(RetryTopicBootstrapper.class));
531-
this.beanFactory.getBean("internalRetryTopicBootstrapper", RetryTopicBootstrapper.class).bootstrapRetryTopic();
529+
new RootBeanDefinition(org.springframework.kafka.retrytopic.RetryTopicBootstrapper.class));
530+
this.beanFactory.getBean("internalRetryTopicBootstrapper",
531+
org.springframework.kafka.retrytopic.RetryTopicBootstrapper.class).bootstrapRetryTopic();
532532
}
533533
}
534534

spring-kafka/src/main/java/org/springframework/kafka/annotation/RetryableTopicAnnotationProcessor.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@
3737
import org.springframework.kafka.retrytopic.RetryTopicConfigurationBuilder;
3838
import org.springframework.kafka.retrytopic.RetryTopicConfigurer;
3939
import org.springframework.kafka.retrytopic.RetryTopicConstants;
40-
import org.springframework.kafka.retrytopic.RetryTopicInternalBeanNames;
4140
import org.springframework.kafka.support.EndpointHandlerMethod;
4241
import org.springframework.retry.annotation.Backoff;
4342
import org.springframework.retry.backoff.ExponentialBackOffPolicy;
@@ -215,7 +214,8 @@ private EndpointHandlerMethod getDltProcessor(Method listenerMethod, Object bean
215214
}
216215
}
217216
try {
218-
return this.beanFactory.getBean(RetryTopicInternalBeanNames.DEFAULT_KAFKA_TEMPLATE_BEAN_NAME,
217+
return this.beanFactory.getBean(
218+
org.springframework.kafka.retrytopic.RetryTopicInternalBeanNames.DEFAULT_KAFKA_TEMPLATE_BEAN_NAME,
219219
KafkaOperations.class);
220220
}
221221
catch (NoSuchBeanDefinitionException ex) {

spring-kafka/src/main/java/org/springframework/kafka/listener/ErrorHandlerAdapter.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2021 the original author or authors.
2+
* Copyright 2021-2022 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -63,6 +63,7 @@ class ErrorHandlerAdapter implements CommonErrorHandler {
6363
this.batchErrorHandler = batchErrorHandler;
6464
}
6565

66+
@SuppressWarnings("deprecation")
6667
@Override
6768
public boolean remainingRecords() {
6869
return this.errorHandler instanceof RemainingRecordsErrorHandler;
@@ -122,6 +123,7 @@ public void handleOtherException(Exception thrownException, Consumer<?, ?> consu
122123
}
123124
}
124125

126+
@SuppressWarnings("deprecation")
125127
@Override
126128
public void handleRecord(Exception thrownException, ConsumerRecord<?, ?> record, Consumer<?, ?> consumer,
127129
MessageListenerContainer container) {

spring-kafka/src/test/java/org/springframework/kafka/core/reactive/ReactiveKafkaProducerTemplateTransactionIntegrationTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,7 @@ public void shouldSendOneRecordTransactionallyViaTemplateAsSenderRecordAndReceiv
269269
.onErrorResume(error -> reactiveKafkaProducerTemplate.transactionManager()
270270
.abort()
271271
.then(Mono.error(error))))
272-
.expectErrorMatches(throwable -> throwable instanceof KafkaException &&
272+
.expectErrorMatches(throwable -> throwable instanceof IllegalStateException &&
273273
throwable.getMessage().equals("TransactionalId reactive.transaction: Invalid transition " +
274274
"attempted from state READY to state ABORTING_TRANSACTION"))
275275
.verify(DEFAULT_VERIFY_TIMEOUT);

0 commit comments

Comments
 (0)