stanislavkozlovski commented on code in PR #19336:
URL: https://github.com/apache/kafka/pull/19336#discussion_r2025355519
##########
clients/src/main/java/org/apache/kafka/clients/consumer/ConsumerConfig.java:
##########
@@ -220,7 +223,9 @@ public class ConsumerConfig extends AbstractConfig {
"partition of the fetch is larger than this limit, the " +
"batch will still be returned to ensure that the consumer can make
progress. The maximum record batch size " +
"accepted by the broker is defined via
<code>message.max.bytes</code> (broker config) or " +
- "<code>max.message.bytes</code> (topic config). See " +
FETCH_MAX_BYTES_CONFIG + " for limiting the consumer request size.";
+ "<code>max.message.bytes</code> (topic config). See " +
FETCH_MAX_BYTES_CONFIG + " for limiting the consumer request size. " +
+ "Consider increasing this limit especially in the cases of remote
storage reads (KIP-405), because currently only " +
Review Comment:
good point
##########
docs/ops.html:
##########
@@ -4169,6 +4170,7 @@ <h4 class="anchor-heading"><a
id="tiered_storage_limitation" class="anchor-link"
<li>Deleting tiered storage enabled topics is required before disabling
tiered storage at the broker level</li>
Review Comment:
done
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
