Skip to content

build(deps): bump kafka.version from 3.1.0 to 7.1.1-ce - #3

Closed
dependabot[bot] wants to merge 1 commit into
masterfrom
dependabot/maven/kafka.version-7.1.1-ce
Closed

build(deps): bump kafka.version from 3.1.0 to 7.1.1-ce#3
dependabot[bot] wants to merge 1 commit into
masterfrom
dependabot/maven/kafka.version-7.1.1-ce

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github May 5, 2022

Copy link
Copy Markdown

Bumps kafka.version from 3.1.0 to 7.1.1-ce.
Updates kafka-clients from 3.1.0 to 7.1.1-ce

Updates kafka-streams from 3.1.0 to 7.1.1-ce

Updates kafka-streams-test-utils from 3.1.0 to 7.1.1-ce

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

Bumps `kafka.version` from 3.1.0 to 7.1.1-ce.

Updates `kafka-clients` from 3.1.0 to 7.1.1-ce

Updates `kafka-streams` from 3.1.0 to 7.1.1-ce

Updates `kafka-streams-test-utils` from 3.1.0 to 7.1.1-ce

---
updated-dependencies:
- dependency-name: org.apache.kafka:kafka-clients
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: org.apache.kafka:kafka-streams
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: org.apache.kafka:kafka-streams-test-utils
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added the dependencies Pull requests that update a dependency file label May 5, 2022
@astubbs astubbs closed this May 20, 2022
@dependabot @github

dependabot Bot commented on behalf of github May 20, 2022

Copy link
Copy Markdown
Author

OK, I won't notify you again about this release, but will get in touch when a new version is available. You can also ignore all major, minor, or patch releases for a dependency by adding an ignore condition with the desired update_types to your config file.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.

@dependabot
dependabot Bot deleted the dependabot/maven/kafka.version-7.1.1-ce branch May 20, 2022 14:18
astubbs added a commit that referenced this pull request Jul 28, 2026
Address Claude review on #63:
- createTopicsBlocking now waits with a generous 60s bound (was
  unbounded) and throws a clear "timed out waiting for the broker to
  create topics ..." message, so a genuinely unresponsive broker fails
  fast/diagnosably instead of hanging until the outer CI timeout. 60s is
  far above the sub-second a healthy broker needs (the old 1s was the
  flake). Constant-named for intent.
- Move the pl.tlinkowski UniLists import out of the org.apache.kafka run
  to restore alphabetical grouping.
- Update inflight.md + docs/solutions to say "60s bound" not "unbounded".

Finding #3 (createTopics(int) now tolerates TopicExistsException) left
as-is: those topic names carry a random suffix so collisions can't occur.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
astubbs added a commit that referenced this pull request Jul 28, 2026
Address Claude review on #63:
- createTopicsBlocking now waits with a generous 60s bound (was
  unbounded) and throws a clear "timed out waiting for the broker to
  create topics ..." message, so a genuinely unresponsive broker fails
  fast/diagnosably instead of hanging until the outer CI timeout. 60s is
  far above the sub-second a healthy broker needs (the old 1s was the
  flake). Constant-named for intent.
- Move the pl.tlinkowski UniLists import out of the org.apache.kafka run
  to restore alphabetical grouping.
- Update inflight.md + docs/solutions to say "60s bound" not "unbounded".

Finding #3 (createTopics(int) now tolerates TopicExistsException) left
as-is: those topic names carry a random suffix so collisions can't occur.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
astubbs added a commit that referenced this pull request Jul 28, 2026
…#63)

* fix(test): consolidate topic creation to kill flaky Integration Tests

BrokerIntegrationTest.ensureTopic duplicated KafkaClientUtils's topic
creation but waited with .get(1, TimeUnit.SECONDS). On a cold/loaded CI
broker, createTopics regularly takes >1s, so the TimeoutException was
rethrown as a hard failure — flaking the *required* Integration Tests
check and blocking otherwise-green PRs (e.g. #56, #61).

Consolidate onto one blocking helper in the canonical util:
- KafkaClientUtils.createTopic(name, partitions) + shared private
  createTopicsBlocking(): waits unbounded (like the other integration
  admin waits) and tolerates only TopicExistsException, propagating any
  other failure instead of swallowing all ExecutionExceptions.
- ensureTopic() and createTopics(int) both delegate to it — one
  implementation, no drift.

Also make the DRY expectation explicit so this doesn't recur:
- AGENTS.md "Testing": search/extend KafkaClientUtils & BrokerIntegration
  Test before adding test helpers; check docs/solutions/ for prior art.
- docs/solutions/test-issues/: learnings entry for the flake + fix.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* review: bound topic-create wait to 60s + fix import ordering

Address Claude review on #63:
- createTopicsBlocking now waits with a generous 60s bound (was
  unbounded) and throws a clear "timed out waiting for the broker to
  create topics ..." message, so a genuinely unresponsive broker fails
  fast/diagnosably instead of hanging until the outer CI timeout. 60s is
  far above the sub-second a healthy broker needs (the old 1s was the
  flake). Constant-named for intent.
- Move the pl.tlinkowski UniLists import out of the org.apache.kafka run
  to restore alphabetical grouping.
- Update inflight.md + docs/solutions to say "60s bound" not "unbounded".

Finding #3 (createTopics(int) now tolerates TopicExistsException) left
as-is: those topic names carry a random suffix so collisions can't occur.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant