projects
/
demos
/
kafka
/
chat
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
refactor: Removed Interface `ChatHomeService`
[demos/kafka/chat]
/
src
/
main
/
java
/
de
/
juplo
/
kafka
/
chat
/
backend
/
persistence
/
inmemory
/
InMemoryChatHomeService.java
diff --git
a/src/main/java/de/juplo/kafka/chat/backend/persistence/inmemory/InMemoryChatHomeService.java
b/src/main/java/de/juplo/kafka/chat/backend/persistence/inmemory/InMemoryChatHomeService.java
index
95a30db
..
1334124
100644
(file)
--- a/
src/main/java/de/juplo/kafka/chat/backend/persistence/inmemory/InMemoryChatHomeService.java
+++ b/
src/main/java/de/juplo/kafka/chat/backend/persistence/inmemory/InMemoryChatHomeService.java
@@
-1,8
+1,6
@@
package de.juplo.kafka.chat.backend.persistence.inmemory;
import de.juplo.kafka.chat.backend.domain.ChatRoom;
package de.juplo.kafka.chat.backend.persistence.inmemory;
import de.juplo.kafka.chat.backend.domain.ChatRoom;
-import de.juplo.kafka.chat.backend.domain.ChatHomeService;
-import de.juplo.kafka.chat.backend.domain.ShardingStrategy;
import lombok.extern.slf4j.Slf4j;
import reactor.core.publisher.Flux;
import reactor.core.publisher.Mono;
import lombok.extern.slf4j.Slf4j;
import reactor.core.publisher.Flux;
import reactor.core.publisher.Mono;
@@
-11,20
+9,17
@@
import java.util.*;
@Slf4j
@Slf4j
-public class InMemoryChatHomeService
implements ChatHomeService
+public class InMemoryChatHomeService
{
{
- private final ShardingStrategy shardingStrategy;
private final Map<UUID, ChatRoom>[] chatrooms;
public InMemoryChatHomeService(
private final Map<UUID, ChatRoom>[] chatrooms;
public InMemoryChatHomeService(
- ShardingStrategy shardingStrategy,
int numShards,
int[] ownedShards,
Flux<ChatRoom> chatroomFlux)
{
log.debug("Creating InMemoryChatHomeService");
int numShards,
int[] ownedShards,
Flux<ChatRoom> chatroomFlux)
{
log.debug("Creating InMemoryChatHomeService");
- this.shardingStrategy = shardingStrategy;
this.chatrooms = new Map[numShards];
Set<Integer> owned = Arrays
.stream(ownedShards)
this.chatrooms = new Map[numShards];
Set<Integer> owned = Arrays
.stream(ownedShards)
@@
-41,8
+36,7
@@
public class InMemoryChatHomeService implements ChatHomeService
chatroomFlux
.filter(chatRoom ->
{
chatroomFlux
.filter(chatRoom ->
{
- int shard = shardingStrategy.selectShard(chatRoom.getId());
- if (owned.contains(shard))
+ if (owned.contains(chatRoom.getShard()))
{
return true;
}
{
return true;
}
@@
-53,35
+47,21
@@
public class InMemoryChatHomeService implements ChatHomeService
}
})
.toStream()
}
})
.toStream()
- .forEach(chatRoom ->
- {
- getChatRoomMapFor(chatRoom).put(chatRoom.getId(), chatRoom);
- });
+ .forEach(chatroom -> chatrooms[chatroom.getShard()].put(chatroom.getId(), chatroom));
}
}
- @Override
- public Mono<ChatRoom> putChatRoom(ChatRoom chatRoom)
+ public void putChatRoom(ChatRoom chatRoom)
{
{
- getChatRoomMapFor(chatRoom).put(chatRoom.getId(), chatRoom);
- return Mono.just(chatRoom);
+ chatrooms[chatRoom.getShard()].put(chatRoom.getId(), chatRoom);
}
}
- @Override
public Mono<ChatRoom> getChatRoom(int shard, UUID id)
{
return Mono.justOrEmpty(chatrooms[shard].get(id));
}
public Mono<ChatRoom> getChatRoom(int shard, UUID id)
{
return Mono.justOrEmpty(chatrooms[shard].get(id));
}
- @Override
public Flux<ChatRoom> getChatRooms(int shard)
{
return Flux.fromStream(chatrooms[shard].values().stream());
}
public Flux<ChatRoom> getChatRooms(int shard)
{
return Flux.fromStream(chatrooms[shard].values().stream());
}
-
-
- private Map<UUID, ChatRoom> getChatRoomMapFor(ChatRoom chatRoom)
- {
- int shard = shardingStrategy.selectShard(chatRoom.getId());
- return chatrooms[shard];
- }
}
}