projects
/
demos
/
kafka
/
chat
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
refactor: Only `LocalJsonFileStorageStrategy` restores `Chatroom`s
[demos/kafka/chat]
/
src
/
main
/
java
/
de
/
juplo
/
kafka
/
chat
/
backend
/
domain
/
Chatroom.java
diff --git
a/src/main/java/de/juplo/kafka/chat/backend/domain/Chatroom.java
b/src/main/java/de/juplo/kafka/chat/backend/domain/Chatroom.java
index
966a28e
..
60f7274
100644
(file)
--- a/
src/main/java/de/juplo/kafka/chat/backend/domain/Chatroom.java
+++ b/
src/main/java/de/juplo/kafka/chat/backend/domain/Chatroom.java
@@
-18,7
+18,8
@@
public class Chatroom
@Getter
private final String name;
private final PersistenceStrategy persistence;
@Getter
private final String name;
private final PersistenceStrategy persistence;
- private final Sinks.Many<Message> sink;
+ private final int bufferSize;
+ private Sinks.Many<Message> sink;
public Chatroom(
UUID id,
public Chatroom(
UUID id,
@@
-29,7
+30,8
@@
public class Chatroom
this.id = id;
this.name = name;
this.persistence = persistence;
this.id = id;
this.name = name;
this.persistence = persistence;
- this.sink = Sinks.many().multicast().onBackpressureBuffer(bufferSize);
+ this.bufferSize = bufferSize;
+ this.sink = createSink();
}
}
@@
-41,7
+43,14
@@
public class Chatroom
{
return persistence
.persistMessage(Message.MessageKey.of(user, id), timestamp, text)
{
return persistence
.persistMessage(Message.MessageKey.of(user, id), timestamp, text)
- .doOnNext(message -> sink.tryEmitNext(message).orThrow());
+ .doOnNext(message ->
+ {
+ Sinks.EmitResult result = sink.tryEmitNext(message);
+ if (result.isFailure())
+ {
+ log.warn("Emitting of message failed with {} for {}", result.name(), message);
+ }
+ });
}
}
@@
-50,9
+59,11
@@
public class Chatroom
return persistence.getMessage(Message.MessageKey.of(username, messageId));
}
return persistence.getMessage(Message.MessageKey.of(username, messageId));
}
- public Flux<Message> listen()
+
synchronized
public Flux<Message> listen()
{
{
- return sink.asFlux();
+ return sink
+ .asFlux()
+ .doOnCancel(() -> sink = createSink()); // Sink hast to be recreated on auto-cancel!
}
public Flux<Message> getMessages()
}
public Flux<Message> getMessages()
@@
-64,4
+75,12
@@
public class Chatroom
{
return persistence.getMessages(first, last);
}
{
return persistence.getMessages(first, last);
}
+
+ private Sinks.Many<Message> createSink()
+ {
+ return Sinks
+ .many()
+ .multicast()
+ .onBackpressureBuffer(bufferSize);
+ }
}
}