Skip to content

Commit

Permalink
review: added comment for config key "subscriber-pool-size"
Browse files Browse the repository at this point in the history
Signed-off-by: Thomas Jaeckle <thomas.jaeckle@bosch.io>
  • Loading branch information
thjaeckle committed Aug 19, 2022
1 parent e2741ef commit c59b3e1
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
8 changes: 7 additions & 1 deletion internal/utils/ddata/src/main/resources/reference.conf
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,13 @@ ditto {
number-of-shards = 5
number-of-shards = ${?DITTO_DDATA_NUMBER_OF_SHARDS}

subscriber-pool-size = 1
# The number of subscribers in each subscriber pool.
# Akka deserializes (remote) messages addressed to 1 actor sequentially, which makes deserialization throughput
# unable to scale with the number of CPUs given to the instance.
# Each instance starts a fixed number of `subscriber-pool-size` subscribers.
# Published signals are sent to each subscriber according to the entity ID.
# This allows signals to different things to be deserialized in parallel.
^ = 1
subscriber-pool-size = ${?akka.remote.artery.advanced.inbound-lanes}
subscriber-pool-size = ${?REMOTE_INBOUND_LANES}
subscriber-pool-size = ${?DITTO_DDATA_SUBSCRIBER_POOL_SIZE}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@
abstract class AbstractSubscriber<T extends SignalWithEntityId<?>> extends AbstractActorWithTimers {

final DittoDiagnosticLoggingAdapter logger = DittoLoggerFactory.getDiagnosticLoggingAdapter(this);

final Class<T> messageClass;
final PubSubTopicExtractor<T> topicExtractor;
final AckExtractor<T> ackExtractor;
Expand Down

0 comments on commit c59b3e1

Please sign in to comment.