package de.juplo.kafka;
+import com.mongodb.client.MongoClient;
import lombok.extern.slf4j.Slf4j;
+import org.apache.kafka.clients.consumer.ConsumerRebalanceListener;
import org.apache.kafka.clients.consumer.ConsumerRecord;
import org.apache.kafka.clients.consumer.KafkaConsumer;
import org.apache.kafka.clients.producer.KafkaProducer;
import org.junit.jupiter.api.*;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.autoconfigure.EnableAutoConfiguration;
+import org.springframework.boot.autoconfigure.mongo.MongoProperties;
import org.springframework.boot.test.autoconfigure.data.mongo.AutoConfigureDataMongo;
import org.springframework.boot.test.context.ConfigDataApplicationContextInitializer;
import org.springframework.boot.test.context.TestConfiguration;
properties = {
"sumup.adder.bootstrap-server=${spring.embedded.kafka.brokers}",
"sumup.adder.topic=" + TOPIC,
- "sumup.adder.commit-interval=1s",
+ "sumup.adder.commit-interval=500ms",
"spring.mongodb.embedded.version=4.4.13" })
@EmbeddedKafka(topics = TOPIC, partitions = PARTITIONS)
@EnableAutoConfiguration
@Autowired
ExecutorService executor;
@Autowired
- PartitionStatisticsRepository partitionStatisticsRepository;
+ MongoClient mongoClient;
@Autowired
- PollIntervalAwareConsumerRebalanceListener rebalanceListener;
+ MongoProperties mongoProperties;
+ @Autowired
+ ConsumerRebalanceListener rebalanceListener;
@Autowired
RecordHandler<K, V> recordHandler;
KafkaConsumer<Bytes, Bytes> offsetConsumer;
EndlessConsumer<K, V> endlessConsumer;
Map<TopicPartition, Long> oldOffsets;
- Map<TopicPartition, Long> newOffsets;
+ Map<TopicPartition, Long> seenOffsets;
Set<ConsumerRecord<K, V>> receivedRecords;
/** Tests methods */
@Test
- void commitsCurrentOffsetsOnSuccess()
+ void commitsCurrentOffsetsOnSuccess() throws Exception
{
int numberOfGeneratedMessages =
recordGenerator.generate(false, false, messageSender);
.untilAsserted(() ->
{
checkSeenOffsetsForProgress();
- compareToCommitedOffsets(newOffsets);
+ assertSeenOffsetsEqualCommittedOffsets(seenOffsets);
});
assertThatExceptionOfType(IllegalStateException.class)
.isThrownBy(() -> endlessConsumer.exitStatus())
.describedAs("Consumer should still be running");
+ endlessConsumer.stop();
recordGenerator.assertBusinessLogic();
}
.until(() -> !endlessConsumer.running());
checkSeenOffsetsForProgress();
- compareToCommitedOffsets(newOffsets);
+ assertSeenOffsetsEqualCommittedOffsets(seenOffsets);
endlessConsumer.start();
await("Consumer failed")
.until(() -> !endlessConsumer.running());
checkSeenOffsetsForProgress();
- compareToCommitedOffsets(newOffsets);
+ assertSeenOffsetsEqualCommittedOffsets(seenOffsets);
assertThat(receivedRecords.size())
.describedAs("Received not all sent events")
.isLessThan(numberOfGeneratedMessages);
.until(() -> !endlessConsumer.running());
checkSeenOffsetsForProgress();
- compareToCommitedOffsets(oldOffsets);
+ assertSeenOffsetsAreBehindCommittedOffsets(seenOffsets);
endlessConsumer.start();
await("Consumer failed")
.pollInterval(Duration.ofSeconds(1))
.until(() -> !endlessConsumer.running());
- checkSeenOffsetsForProgress();
- compareToCommitedOffsets(oldOffsets);
- assertThat(receivedRecords.size())
- .describedAs("Received not all sent events")
- .isLessThan(numberOfGeneratedMessages);
+ assertSeenOffsetsAreBehindCommittedOffsets(seenOffsets);
assertThatNoException()
.describedAs("Consumer should not be running")
/** Helper methods for the verification of expectations */
- void compareToCommitedOffsets(Map<TopicPartition, Long> offsetsToCheck)
+ void assertSeenOffsetsEqualCommittedOffsets(Map<TopicPartition, Long> offsetsToCheck)
{
doForCurrentOffsets((tp, offset) ->
{
Long expected = offsetsToCheck.get(tp) + 1;
- log.debug("Checking, if the offset for {} is {}", tp, expected);
+ log.debug("Checking, if the offset {} for {} is exactly {}", offset, tp, expected);
assertThat(offset)
.describedAs("Committed offset corresponds to the offset of the consumer")
.isEqualTo(expected);
});
}
+ void assertSeenOffsetsAreBehindCommittedOffsets(Map<TopicPartition, Long> offsetsToCheck)
+ {
+ List<Boolean> isOffsetBehindSeen = new LinkedList<>();
+
+ doForCurrentOffsets((tp, offset) ->
+ {
+ Long expected = offsetsToCheck.get(tp) + 1;
+ log.debug("Checking, if the offset {} for {} is at most {}", offset, tp, expected);
+ assertThat(offset)
+ .describedAs("Committed offset corresponds to the offset of the consumer")
+ .isLessThanOrEqualTo(expected);
+ isOffsetBehindSeen.add(offset < expected);
+ });
+
+ assertThat(isOffsetBehindSeen.stream().reduce(false, (result, next) -> result | next))
+ .describedAs("Committed offsets are behind seen offsets")
+ .isTrue();
+ }
+
void checkSeenOffsetsForProgress()
{
// Be sure, that some messages were consumed...!
partitions().forEach(tp ->
{
Long oldOffset = oldOffsets.get(tp) + 1;
- Long newOffset = newOffsets.get(tp) + 1;
+ Long newOffset = seenOffsets.get(tp) + 1;
if (!oldOffset.equals(newOffset))
{
log.debug("Progress for {}: {} -> {}", tp, oldOffset, newOffset);
void seekToEnd()
{
offsetConsumer.assign(partitions());
+ offsetConsumer.seekToEnd(partitions());
partitions().forEach(tp ->
{
+ // seekToEnd() works lazily: it only takes effect on poll()/position()
Long offset = offsetConsumer.position(tp);
log.info("New position for {}: {}", tp, offset);
- Integer partition = tp.partition();
- StateDocument document =
- partitionStatisticsRepository
- .findById(partition.toString())
- .orElse(new StateDocument(partition));
- document.offset = offset;
- partitionStatisticsRepository.save(document);
});
+ // The new positions must be commited!
+ offsetConsumer.commitSync();
offsetConsumer.unsubscribe();
}
void doForCurrentOffsets(BiConsumer<TopicPartition, Long> consumer)
{
- partitions().forEach(tp ->
- {
- String partition = Integer.toString(tp.partition());
- Optional<Long> offset = partitionStatisticsRepository.findById(partition).map(document -> document.offset);
- consumer.accept(tp, offset.orElse(0l));
- });
+ offsetConsumer.assign(partitions());
+ partitions().forEach(tp -> consumer.accept(tp, offsetConsumer.position(tp)));
+ offsetConsumer.unsubscribe();
}
List<TopicPartition> partitions()
props.put("value.deserializer", BytesDeserializer.class.getName());
offsetConsumer = new KafkaConsumer<>(props);
+ mongoClient.getDatabase(mongoProperties.getDatabase()).drop();
seekToEnd();
oldOffsets = new HashMap<>();
- newOffsets = new HashMap<>();
+ seenOffsets = new HashMap<>();
receivedRecords = new HashSet<>();
doForCurrentOffsets((tp, offset) ->
{
oldOffsets.put(tp, offset - 1);
- newOffsets.put(tp, offset - 1);
+ seenOffsets.put(tp, offset - 1);
});
TestRecordHandler<K, V> captureOffsetAndExecuteTestHandler =
@Override
public void onNewRecord(ConsumerRecord<K, V> record)
{
- newOffsets.put(
+ seenOffsets.put(
new TopicPartition(record.topic(), record.partition()),
record.offset());
receivedRecords.add(record);
{
try
{
- endlessConsumer.stop();
testRecordProducer.close();
offsetConsumer.close();
}
projects / demos / kafka / training / blobdiff