X-Git-Url: https://juplo.de/gitweb/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fde%2Fjuplo%2Fkafka%2Fchat%2Fbackend%2Fpersistence%2Fkafka%2FKafkaChatHomeService.java;h=912295d657db00fd0be4c38e06db7db8f6216968;hb=1fb148e961483be6aeb1170ff0673a2682f9ad01;hp=4460432254379ca6ca0a420db8e4d795f03aa4d7;hpb=394aa7848abcfc7b4510cd2688c00fce01b3c225;p=demos%2Fkafka%2Fchat diff --git a/src/main/java/de/juplo/kafka/chat/backend/persistence/kafka/KafkaChatHomeService.java b/src/main/java/de/juplo/kafka/chat/backend/persistence/kafka/KafkaChatHomeService.java index 44604322..912295d6 100644 --- a/src/main/java/de/juplo/kafka/chat/backend/persistence/kafka/KafkaChatHomeService.java +++ b/src/main/java/de/juplo/kafka/chat/backend/persistence/kafka/KafkaChatHomeService.java @@ -2,27 +2,23 @@ package de.juplo.kafka.chat.backend.persistence.kafka; import de.juplo.kafka.chat.backend.domain.ChatHomeService; import de.juplo.kafka.chat.backend.domain.ChatRoom; +import de.juplo.kafka.chat.backend.domain.Message; import de.juplo.kafka.chat.backend.domain.ShardNotOwnedException; +import de.juplo.kafka.chat.backend.persistence.KafkaLikeShardingStrategy; 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.clients.producer.Producer; +import org.apache.kafka.clients.producer.ProducerRecord; import org.apache.kafka.common.TopicPartition; import reactor.core.publisher.Flux; import reactor.core.publisher.Mono; -import java.time.Duration; -import java.time.ZoneId; +import java.time.*; import java.util.*; import java.util.concurrent.ExecutorService; -import java.util.concurrent.TimeUnit; -import java.util.concurrent.TimeoutException; -import java.util.concurrent.locks.Lock; -import java.util.concurrent.locks.ReadWriteLock; -import java.util.concurrent.locks.ReentrantLock; -import java.util.concurrent.locks.ReentrantReadWriteLock; @Slf4j @@ -33,12 +29,15 @@ public class KafkaChatHomeService implements ChatHomeService, Runnable, Consumer private final Producer producer; private final String topic; private final ZoneId zoneId; - // private final long[] offsets; Erst mal immer alles neu einlesen + private final int numShards; private final boolean[] isShardOwned; + private final long[] currentOffset; + private final long[] nextOffset; private final Map[] chatRoomMaps; - private final ReadWriteLock lock = new ReentrantReadWriteLock(); + private final KafkaLikeShardingStrategy shardingStrategy; private boolean running; + private volatile boolean loadInProgress; public KafkaChatHomeService( @@ -55,50 +54,36 @@ public class KafkaChatHomeService implements ChatHomeService, Runnable, Consumer this.producer = producer; this.topic = topic; this.zoneId = zoneId; - // this.offsets = new long[numShards]; - // for (int i=0; i< numShards; i++) - // { - // this.offsets[i] = 0l; - // } + this.numShards = numShards; this.isShardOwned = new boolean[numShards]; + this.currentOffset = new long[numShards]; + this.nextOffset = new long[numShards]; this.chatRoomMaps = new Map[numShards]; + this.shardingStrategy = new KafkaLikeShardingStrategy(numShards); } @Override public void onPartitionsAssigned(Collection partitions) { - try - { - lock.writeLock().lock(); - - consumer.endOffsets(partitions).forEach((topicPartition, currentOffset) -> - { - if (!topicPartition.topic().equals(topic)) - { - log.warn("Ignoring partition from unwanted topic: {}", topicPartition); - return; - } + loadInProgress = true; - int partition = topicPartition.partition(); - long unseenOffset = 0; // offsets[partition]; + consumer.endOffsets(partitions).forEach((topicPartition, currentOffset) -> + { + int partition = topicPartition.partition(); + isShardOwned[partition] = true; + this.currentOffset[partition] = currentOffset; - log.info( - "Loading messages from partition {}: start-offset={} -> current-offset={}", - partition, - unseenOffset, - currentOffset); + log.info( + "Partition assigned: {} - loading messages: next={} -> current={}", + partition, + nextOffset[partition], + currentOffset); - // TODO: reuse! Nicht immer alles neu laden, sondern erst ab offsets[partition]! - consumer.seek(topicPartition, unseenOffset); - }); + consumer.seek(topicPartition, nextOffset[partition]); + }); - consumer.resume(partitions); - } - finally - { - lock.writeLock().unlock(); - } + consumer.resume(partitions); } @Override @@ -106,21 +91,16 @@ public class KafkaChatHomeService implements ChatHomeService, Runnable, Consumer { partitions.forEach(topicPartition -> { - if (!topicPartition.topic().equals(topic)) - { - log.warn("Ignoring partition from unwanted topic: {}", topicPartition); - return; - } - int partition = topicPartition.partition(); - // long unseenOffset = offsets[partition]; TODO: Offset merken...? + isShardOwned[partition] = false; + log.info("Partition revoked: {} - next={}", partition, nextOffset[partition]); }); - log.info("Revoked partitions: {}", partitions); } @Override public void onPartitionsLost(Collection partitions) { + log.warn("Lost partitions: {}, partitions"); // TODO: Muss auf den Verlust anders reagiert werden? onPartitionsRevoked(partitions); } @@ -139,51 +119,106 @@ public class KafkaChatHomeService implements ChatHomeService, Runnable, Consumer ConsumerRecords records = consumer.poll(Duration.ofMinutes(5)); log.info("Fetched {} messages", records.count()); - for (ConsumerRecord record : records) + if (loadInProgress) { + for (ConsumerRecord record : records) + { + UUID chatRoomId = UUID.fromString(record.key()); + MessageTo messageTo = record.value(); + + Message.MessageKey key = Message.MessageKey.of(messageTo.getUser(), messageTo.getId()); + + Instant instant = Instant.ofEpochSecond(record.timestamp()); + LocalDateTime timestamp = LocalDateTime.ofInstant(instant, zoneId); + Message message = new Message(key, record.offset(), timestamp, messageTo.getText()); + + ChatRoom chatRoom = chatRoomMaps[record.partition()].get(chatRoomId); + KafkaChatRoomService kafkaChatRoomService = + (KafkaChatRoomService) chatRoom.getChatRoomService(); + + kafkaChatRoomService.persistMessage(message); + } + } + else + { + if (!records.isEmpty()) + { + throw new IllegalStateException("All owned partitions should be paused, when no load is in progress!"); + } } } } } + Mono sendMessage( + UUID chatRoomId, + Message.MessageKey key, + LocalDateTime timestamp, + String text) + { + int shard = this.shardingStrategy.selectShard(chatRoomId); + TopicPartition tp = new TopicPartition(topic, shard); + ZonedDateTime zdt = ZonedDateTime.of(timestamp, zoneId); + return Mono.create(sink -> + { + ProducerRecord record = + new ProducerRecord<>( + tp.topic(), + tp.partition(), + zdt.toEpochSecond(), + chatRoomId.toString(), + MessageTo.of(key.getUsername(), key.getMessageId(), text)); + + producer.send(record, ((metadata, exception) -> + { + if (metadata != null) + { + // On successful send + Message message = new Message(key, metadata.offset(), timestamp, text); + log.info("Successfully send message {}", message); + sink.success(message); + } + else + { + // On send-failure + log.error( + "Could not send message for chat-room={}, key={}, timestamp={}, text={}: {}", + chatRoomId, + key, + timestamp, + text, + exception); + sink.error(exception); + } + })); + }); + } + + @Override public Mono getChatRoom(int shard, UUID id) { - if (lock.readLock().tryLock()) + if (loadInProgress) { - try - { - return Mono.justOrEmpty(chatRoomMaps[shard].get(id)); - } - finally - { - lock.readLock().unlock(); - } + throw new ShardNotOwnedException(shard); } else { - throw new ShardNotOwnedException(shard); + return Mono.justOrEmpty(chatRoomMaps[shard].get(id)); } } @Override public Flux getChatRooms(int shard) { - if (lock.readLock().tryLock()) + if (loadInProgress) { - try - { - return Flux.fromStream(chatRoomMaps[shard].values().stream()); - } - finally - { - lock.readLock().unlock(); - } + throw new ShardNotOwnedException(shard); } else { - throw new ShardNotOwnedException(shard); + return Flux.fromStream(chatRoomMaps[shard].values().stream()); } } }