Skip to content

KAFKA-2526: command line --producer-property wins #16492

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

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

Conversation

jkt628
Copy link

@jkt628 jkt628 commented Jun 30, 2024

apply --producer-property settings last to override any, including default, values.

tested manually with value.serializer. the extant test for client.id is unaffected.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@jkt628
Copy link
Author

jkt628 commented Jul 1, 2024

test failures seem very much UNRELATED to this change.

Copy link

This PR is being marked as stale since it has not had any activity in 90 days. If you
would like to keep this PR alive, please leave a comment asking for a review. If the PR has
merge conflicts, update it with the latest from the base branch.

If you are having difficulty finding a reviewer, please reach out on the [mailing list](https://kafka.apache.org/contact).

If this PR is no longer valid or desired, please feel free to close it. If no activity occurs in the next 30 days, it will be automatically closed.

@github-actions github-actions bot added the stale Stale PRs label Jan 16, 2025
@jkt628
Copy link
Author

jkt628 commented Jan 16, 2025

unstale

@jkt628
Copy link
Author

jkt628 commented Jan 16, 2025

./gradlew clients:test -PmaxParallelForks=2 passes but i cannot run the full suite due to OOM.

@github-actions github-actions bot removed the stale Stale PRs label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant