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

Update librdkafka to version 1.4.2 #11256

Merged
merged 2 commits into from
May 31, 2020

Conversation

filimonov
Copy link
Contributor

@filimonov filimonov commented May 28, 2020

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Update librdkafka to version 1.4.2

Detailed description / Documentation draft:
I think it closes #5637 (See also: confluentinc/librdkafka#2183 (comment) )

@blinkov blinkov added the pr-improvement Pull request with some product improvements label May 28, 2020
@filimonov
Copy link
Contributor Author

It looks like newer version behave a bit different

"2020.05.28 22:40:01.633450 [ 67 ] {} <Warning> StorageKafka (kafka): [thrd:main]: Consumer group session timed out (in join-state started) after 10498 ms without a successful response from the group coordinator (broker 1, last error was Success): revoking assignment and rejoining group"

It looks related to confluentinc/librdkafka#2631
So now it leaves the group preventing further commits. Acceptable.

Will try to adjust the test.

@alexey-milovidov
Copy link
Member

Logical error:

2020.05.29 02:58:44.483982 [ 207 ] {} <Trace> test_3bd5c8.alter_table2: Renaming temporary part tmp_mut_-2_0_0_1_9 to -2_0_0_1_9.
2020.05.29 02:58:44.485340 [ 207 ] {} <Error> : Logical error: 'Part -2_0_0_1_9 intersects next part -2_0_0_2 (state Committed). It is a bug.'.

In test 00992_system_parts_race_condition_zookeeper

See #10066

@alexey-milovidov alexey-milovidov self-assigned this May 31, 2020
@alexey-milovidov alexey-milovidov merged commit fb1a2fa into ClickHouse:master May 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-improvement Pull request with some product improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update rdkafka when it gets support for zstd-compressed topics
3 participants