Skip to content

Compare feature flags between 24-4 and 25-1 #17536

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

Closed
maximyurchuk opened this issue Apr 22, 2025 · 3 comments
Closed

Compare feature flags between 24-4 and 25-1 #17536

maximyurchuk opened this issue Apr 22, 2025 · 3 comments
Assignees

Comments

@maximyurchuk
Copy link
Collaborator

maximyurchuk commented Apr 22, 2025

Есть дефолты фича флагов:
25-1: https://github.com/ydb-platform/ydb/blob/stable-25-1/ydb/core/protos/feature_flags.proto
24-4: https://github.com/ydb-platform/ydb/blob/stable-24-4/ydb/core/protos/feature_flags.proto

Нужно найти фича флаги которые были false в 24-4 и стали true в 25-1

После этого сходить к владельцам этих фича флагов и убедиться что:

  • это переключение осознанно
  • фича флаги были включены на проде (то есть дефолт переключается не "вслепую")
@galnat
Copy link
Collaborator

galnat commented Apr 22, 2025

Переменные, у которых изменились значения по умолчанию:

Оставляем

  • optional bool EnableAlterDatabaseCreateHiveFirst = 82 [default = true]; - "Правильное" создание системных таблеток тенанта. Автор - Иван Челюбеев. Проверялся на ydb-ru-prestable какое-то время. Оставляем.

  • optional bool EnableImplicitQueryParameterTypes = 93 [default = true]; так оказалось что в 25-1 фича включена по дефолту, раньше нигде не включалась, автор - Виталий Гриднев. Откатить это уже не будет просто, потому что это выкатилось на prestable и это уже заиспользовали. Оставляем.

  • optional bool EnableDataShardInMemoryStateMigration = 159 [default = true]; -
    Миграция состояния памяти между даташардами на рестартах, обратно совместимо, включено по-умолчанию (фича флаг на случай необходимости аварийного выключения). Автор - Алексей Борзенков. Оставляем.

Выключаем

  • optional bool EnableTempTables = 102 [default = true]; - включил 8.11.24 Николай Шумков 645eb31. Это новая функция, не влияет на обычный путь запроса. Виталий Гриднев предлагает оставить - можно и выключить в целом, но с другой стороны ревльно протестировать без пользователя не очень получится, этим никто не пользуется и логика не задействуется. Давайте выключим и включим на конкретном прод кластере для проверки Жду PR от Виталия Гриднева.

  • optional bool EnableResourcePools = 129 [default = true]; Выключаем в 25-1-1, проверяем на прод кластере и включим по дефолту в 25-1-2. Выключаем Disabled resource pools #17715

  • optional bool EnablePgSyntax = 139 [default = true]; - включили 5-го сентября 7f64b2d#diff-8f86348e2bc87be6446c68e7ecf1bbcf3a5655782c728ff52baadad718cb7dbd Николай Шумков. Disable EnablePgSyntax by default in 25-1 #17697
    Выключаем disable pg syntax feature flag #17658

  • optional bool EnableParameterizedDecimal = 145 [default = true]; - Decimal с произвольными параметрами. Код параметризованных Decimal уже работает в 24.3 и 24.4. В 25.1 будет разрешено создавать колонки с произвольными параметрами, отличными от значений по умолчанию. Это фичефлаг не включался на прод других кластерах& Включал Саша 1.11 - aec6644. Давайте выключим и включим на конкретном прод кластере для проверки. Disable EnableParameterizedDecimal by default in 25-1 #17698

  • [] optional bool EnableSeparationComputeActorsFromRead = 90 [default = true]; включил 10.07.2024 Женя Зверев 0f81829, флаг на стыке с Виталием Гридневым. Нужен для аналитических баз ( ну и работает сугубо для них) , там и тестировался. Ломало обратную совместимость. Выключаем.

@maximyurchuk
Copy link
Collaborator Author

EnableTempTables выключили
#17732
#17753

EnableResourcePools выключили
#17715
#17716

EnablePgSyntax выключили
#17658
#17657

EnableParameterizedDecimal выключили
#17707
#17708

Все из секции "Выключаем" в сообщении выше выключили

@maximyurchuk
Copy link
Collaborator Author

maximyurchuk commented Apr 28, 2025

По EnableSeparationComputeActorsFromRead -- вероятно тоже надо выключить,
#17778
#17779

Больше инфы у @swalrus1
Проблема вот #17462

@galnat galnat closed this as completed Apr 28, 2025
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

No branches or pull requests

2 participants