Update to s-k 1.1.0

Listener changes require a bump of the s-i-k version.

This is an initial commit - we will add support for new features (such as batch
payloads in future commits).

Polishing

* Upgrade to SI-4.3.x
* Use `Assert.instanceOf()`
This commit is contained in:
Gary Russell
2016-09-06 17:08:33 -04:00
committed by Artem Bilan
parent 8662c10232
commit 0cf61af620
2 changed files with 22 additions and 5 deletions

View File

@@ -23,11 +23,12 @@ import org.springframework.integration.endpoint.MessageProducerSupport;
import org.springframework.kafka.listener.AbstractMessageListenerContainer;
import org.springframework.kafka.listener.AcknowledgingMessageListener;
import org.springframework.kafka.listener.adapter.FilteringAcknowledgingMessageListenerAdapter;
import org.springframework.kafka.listener.adapter.MessagingMessageListenerAdapter;
import org.springframework.kafka.listener.adapter.RecordFilterStrategy;
import org.springframework.kafka.listener.adapter.RecordMessagingMessageListenerAdapter;
import org.springframework.kafka.listener.adapter.RetryingAcknowledgingMessageListenerAdapter;
import org.springframework.kafka.support.Acknowledgment;
import org.springframework.kafka.support.converter.MessageConverter;
import org.springframework.kafka.support.converter.RecordMessageConverter;
import org.springframework.messaging.Message;
import org.springframework.retry.RecoveryCallback;
import org.springframework.retry.support.RetryTemplate;
@@ -48,7 +49,7 @@ public class KafkaMessageDrivenChannelAdapter<K, V> extends MessageProducerSuppo
private final AbstractMessageListenerContainer<K, V> messageListenerContainer;
private final MessagingMessageListenerAdapter<K, V> listener = new IntegrationMessageListener();
private final RecordMessagingMessageListenerAdapter<K, V> listener = new IntegrationMessageListener();
private RecordFilterStrategy<K, V> recordFilterStrategy;
@@ -68,7 +69,23 @@ public class KafkaMessageDrivenChannelAdapter<K, V> extends MessageProducerSuppo
this.messageListenerContainer.setAutoStartup(false);
}
/**
* Set the message converter; must be a {@link RecordMessageConverter}.
* @param messageConverter the converter.
* @deprecated in favor of {@link #setRecordMessageConverter(RecordMessageConverter)}.
*/
@Deprecated
public void setMessageConverter(MessageConverter messageConverter) {
Assert.isInstanceOf(RecordMessageConverter.class, messageConverter);
this.listener.setMessageConverter((RecordMessageConverter) messageConverter);
}
/**
* Set the message converter to use with a record-based consumer.
* @param messageConverter the converter.
* @since 2.1
*/
public void setRecordMessageConverter(RecordMessageConverter messageConverter) {
this.listener.setMessageConverter(messageConverter);
}
@@ -185,10 +202,10 @@ public class KafkaMessageDrivenChannelAdapter<K, V> extends MessageProducerSuppo
return getPhase();
}
private class IntegrationMessageListener extends MessagingMessageListenerAdapter<K, V> {
private class IntegrationMessageListener extends RecordMessagingMessageListenerAdapter<K, V> {
IntegrationMessageListener() {
super(null);
super(null, null);
}
@Override

View File

@@ -68,7 +68,7 @@ public class MessageDrivenAdapterTests {
QueueChannel out = new QueueChannel();
adapter.setOutputChannel(out);
adapter.afterPropertiesSet();
adapter.setMessageConverter(new MessagingMessageConverter() {
adapter.setRecordMessageConverter(new MessagingMessageConverter() {
@Override
public Message<?> toMessage(ConsumerRecord<?, ?> record, Acknowledgment acknowledgment, Type type) {