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

Add SETTINGS clause for Kafka storage engine #2781

Merged
merged 1 commit into from
Aug 3, 2018
Merged

Add SETTINGS clause for Kafka storage engine #2781

merged 1 commit into from
Aug 3, 2018

Conversation

Amper
Copy link
Contributor

@Amper Amper commented Aug 1, 2018

Собирался добавить ещё один параметр в KafkaEngine и обратил внимание на две вещи:

  1. Одно из последних изменений (Add row_delimiter argument to StorageKafka. #2701) сломало обратную совместимость, так как параметр был добавлен не последним, а предпоследним.
  2. При этом понятно, почему так было сделано. Но, тем не менее, список параметров движка стал монструозным и неудобным - например, при текущем варианте записи невозможно нормально задать количество консюмеров без задания всех параметров (даже тех, что не нужны).

Поэтому я реализовал поддержку SETTINGS-секции для задания параметров движка Kafka. При этом обратную совместимость я не сломал - поддержка старого варианта никуда не делась. Более того, варианты можно комбинировать между собой в пределах разумного (один и тот же параметр можно задать либо в аргументах движка, либо в SETTINGS, но не одновременно в двух местах).

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

@Amper Amper changed the title Added SETTINGS clause for Kafka storage engine Add SETTINGS clause for Kafka storage engine Aug 1, 2018
@alexey-milovidov
Copy link
Member

Спасибо! Действительно, так гораздо лучше.

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.

2 participants