hachikuji commented on a change in pull request #11616:
URL: https://github.com/apache/kafka/pull/11616#discussion_r801172935



##########
File path: config/producer.properties
##########
@@ -23,7 +23,7 @@ bootstrap.servers=localhost:9092
 # specify the compression codec for all data generated: none, gzip, snappy, 
lz4, zstd
 compression.type=none
 
-# name of the partitioner class for partitioning events; default partition 
spreads data randomly
+# name of the partitioner class for partitioning events; default partitioner 
will try sticking to a partition until the batch is full

Review comment:
       Do we use "events" consistently? Seems like "records" would be more in 
keeping with the APIs. For the latter half, how about this?
   
   > The default uses "sticky" partitioning logic, which attempts to fill 
batches sent to each broker in order to improve throughput.
   
   




-- 
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


Reply via email to