You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by gu...@apache.org on 2016/02/02 18:38:54 UTC
kafka git commit: KAFKA-3170;
Set default fetch_min_bytes in new consumer to 1
Repository: kafka
Updated Branches:
refs/heads/trunk c4f32c53e -> c97a75d98
KAFKA-3170; Set default fetch_min_bytes in new consumer to 1
Set default to 1 instead of 1024, this matches the existing doc and feels like a better default value. Have run the unit tests with the change.
Author: Rajini Sivaram <ra...@googlemail.com>
Reviewers: Jason Gustafson, Ismael Juma, Grant Henke, Guozhang Wang
Closes #832 from rajinisivaram/KAFKA-3170
Project: http://git-wip-us.apache.org/repos/asf/kafka/repo
Commit: http://git-wip-us.apache.org/repos/asf/kafka/commit/c97a75d9
Tree: http://git-wip-us.apache.org/repos/asf/kafka/tree/c97a75d9
Diff: http://git-wip-us.apache.org/repos/asf/kafka/diff/c97a75d9
Branch: refs/heads/trunk
Commit: c97a75d985e42a9d1b1be0352240c45e0b0d2978
Parents: c4f32c5
Author: Rajini Sivaram <ra...@googlemail.com>
Authored: Tue Feb 2 09:38:52 2016 -0800
Committer: Guozhang Wang <wa...@gmail.com>
Committed: Tue Feb 2 09:38:52 2016 -0800
----------------------------------------------------------------------
.../java/org/apache/kafka/clients/consumer/ConsumerConfig.java | 2 +-
docs/upgrade.html | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kafka/blob/c97a75d9/clients/src/main/java/org/apache/kafka/clients/consumer/ConsumerConfig.java
----------------------------------------------------------------------
diff --git a/clients/src/main/java/org/apache/kafka/clients/consumer/ConsumerConfig.java b/clients/src/main/java/org/apache/kafka/clients/consumer/ConsumerConfig.java
index 9355b08..df192b9 100644
--- a/clients/src/main/java/org/apache/kafka/clients/consumer/ConsumerConfig.java
+++ b/clients/src/main/java/org/apache/kafka/clients/consumer/ConsumerConfig.java
@@ -226,7 +226,7 @@ public class ConsumerConfig extends AbstractConfig {
CommonClientConfigs.RECEIVE_BUFFER_DOC)
.define(FETCH_MIN_BYTES_CONFIG,
Type.INT,
- 1024,
+ 1,
atLeast(0),
Importance.HIGH,
FETCH_MIN_BYTES_DOC)
http://git-wip-us.apache.org/repos/asf/kafka/blob/c97a75d9/docs/upgrade.html
----------------------------------------------------------------------
diff --git a/docs/upgrade.html b/docs/upgrade.html
index 3244a2c..ba59cc0 100644
--- a/docs/upgrade.html
+++ b/docs/upgrade.html
@@ -58,6 +58,7 @@
<li> The new broker id generation feature can be disable by setting broker.id.generation.enable to false. </li>
<li> Configuration parameter log.cleaner.enable is now true by default. This means topics with a cleanup.policy=compact will now be compacted by default, and 128 MB of heap will be allocated to the cleaner process via log.cleaner.dedupe.buffer.size. You may want to review log.cleaner.dedupe.buffer.size and the other log.cleaner configuration values based on your usage of compacted topics. </li>
<li> MirrorMakerMessageHandler no longer exposes <em>handle(record: MessageAndMetadata[Array[Byte], Array[Byte]])</em> method as it was never called. </li>
+ <li> Default value of configuration parameter fetch.min.bytes for the new consumer is now 1 by default. </li>
</ul>
<h5>Deprecations in 0.9.0.0</h5>