Skip to content

Commit 44683ac

Browse files
authored
Merge pull request #17 from rh-marketingops/consumer-cfg-hotfix
fix bad consumer value
2 parents 0b40a15 + 0e34144 commit 44683ac

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

fluvii/consumer/config.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,9 @@ class ConsumerConfig(KafkaConfigBase, BaseSettings):
1717
batch_consume_max_time_seconds: Optional[int] = 10
1818
batch_consume_store_messages: bool = False
1919
heartbeat_timeout_ms: int = 4 * 60 // 2 * 1_000 # TODO document that this pairs with timeout_minutes
20+
message_singleton_max_mb: int = 2
2021
message_batch_max_mb: int = 5
2122
message_queue_max_mb: int = 20
22-
message_total_max_mb: int = 2
2323
poll_timeout_seconds: int = 5
2424
timeout_minutes: int = 4 # TODO document that this pairs with heartbeat_timeout_ms
2525

@@ -31,10 +31,10 @@ def as_client_dict(self):
3131
return {
3232
"auto.commit.interval.ms": self.auto_commit_interval_seconds * 1_000,
3333
"auto.offset.reset": self.auto_offset_reset,
34-
"fetch.max.bytes": self.message_total_max_mb * (2 ** 20),
34+
"fetch.max.bytes": self.message_batch_max_mb * (2 ** 20),
3535
"heartbeat.interval.ms": (self.heartbeat_timeout_ms // 5) - ms_tolerance, # 5 failed heartbeats == bad consumer.
3636
"max.poll.interval.ms": self.timeout_minutes * 60_000, # Max time between poll() calls before considered dead.
37-
"message.max.bytes": self.message_batch_max_mb * (2 ** 20),
37+
"message.max.bytes": self.message_singleton_max_mb * (2 ** 20),
3838
"queued.max.messages.kbytes": self.message_queue_max_mb * (2 ** 10),
3939
"session.timeout.ms": self.heartbeat_timeout_ms, # need at least 1 heartbeat within "session" time to be considered alive;
4040
}

setup.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323

2424
setuptools.setup(
2525
name="fluvii",
26-
version="0.2.5",
26+
version="0.2.6",
2727
description="A simple Kafka streams implementation in Python using confluent-kafka-python",
2828
packages=packages,
2929
install_requires=install_requires,

0 commit comments

Comments
 (0)