Skip to content

Bump cppkafka (includes support for OAUTHBEARER token refresh callback)#100657

Merged
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
azat:bump-cppkafka
Mar 29, 2026
Merged

Bump cppkafka (includes support for OAUTHBEARER token refresh callback)#100657
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
azat:bump-cppkafka

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Mar 24, 2026

Sync cppkafka submodule properly to include all changes

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@azat
Bump cppkafka (includes cppkafka/pull/5)
1f30ba6 
Sync cppkafka submodule properly to include all changes
@azat azat mentioned this pull request Mar 24, 2026
Merged
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Mar 24, 2026 •
edited
Loading

Workflow [PR], commit [1f30ba6]

Summary: ✅

AI Review

Summary

This PR updates the contrib/cppkafka submodule from 99ee856 to 8cc2f31, pulling upstream changes that add OAUTHBEARER token refresh callback support in cppkafka and related docs/examples. I did not find correctness, safety, compatibility, or performance issues in the ClickHouse-side change (a pure submodule pointer update), and the PR metadata category is consistent for this scope.

ClickHouse Rules
Item Status Notes
Deletion logging ➖
Serialization versioning ➖
Core-area scrutiny ➖
No test removal ✅
Experimental gate ➖
No magic constants ➖
Backward compatibility ✅
SettingsChangesHistory.cpp ➖
PR metadata quality ✅
Safe rollout ✅
Compilation time ✅
Final Verdict
  • Status: ✅ Approve

@clickhouse-gh clickhouse-gh bot added pr-not-for-changelog This PR should not be mentioned in the changelog submodule changed At least one submodule changed in this PR. labels Mar 24, 2026
@azat azat changed the title Bump cppkafka (includes cppkafka/pull/5) Bump cppkafka (includes support for OAUTHBEARER token refresh callback) Mar 24, 2026
@alexey-milovidov alexey-milovidov self-assigned this Mar 29, 2026
@alexey-milovidov alexey-milovidov merged commit 419bf2c into ClickHouse:master Mar 29, 2026
152 of 153 checks passed
@robot-ch-test-poll robot-ch-test-poll added the pr-synced-to-cloud The PR is synced to the cloud repo label Mar 29, 2026
Desel72 pushed a commit to Desel72/ClickHouse that referenced this pull request Mar 30, 2026
@alexey-milovidov
Merge pull request ClickHouse#100657 from azat/bump-cppkafka
c32923f 
Bump cppkafka (includes support for OAUTHBEARER token refresh callback)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexey-milovidov alexey-milovidov alexey-milovidov approved these changes

Assignees

@alexey-milovidov alexey-milovidov

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo submodule changed At least one submodule changed in this PR.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@azat @alexey-milovidov @robot-ch-test-poll