Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove log.flush.interval.messages from Kafka server.properties #55

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Vanlightly
Copy link
Contributor

Apache Kafka flush behavior can be set from the driver config so it is best to leave log.flush.interval.messages out of server properties. Many users who benchmark Kafka from this copy of OMB may leave out the 'flush.messages' driver config and not realise that Kafka will be flushing on each message batch in any case.

Apache Kafka flush behavior can be set from the driver config so
it is best to leave log.flush.interval.messages out of server properties. Many users who benchmark Kafka from this copy of OMB may leave out
the 'flush.messages' driver config and not realise that Kafka will be
flushing on each message batch in any case.
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@mattschumpert
Copy link

fyi @travisdowns

@tmgstevens
Copy link
Contributor

Hi @Vanlightly thanks for joining the Redpanda community! Please could you sign the CLA so that we can merge this please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants