Enable finalizeByKey by default in kqp (#7559)

lll-phill-lll · lll-phill-lll · commit 29e398315194 · 2024-09-12T10:37:40.000Z
diff --git a/ydb/core/kqp/provider/yql_kikimr_settings.cpp b/ydb/core/kqp/provider/yql_kikimr_settings.cpp
@@ -145,7 +145,7 @@ bool TKikimrSettings::HasOptEnableOlapProvideComputeSharding() const {
 }
 
 bool TKikimrSettings::HasOptUseFinalizeByKey() const {
-    return GetOptionalFlagValue(OptUseFinalizeByKey.Get()) != EOptionalFlag::Disabled;
+    return GetFlagValue(OptUseFinalizeByKey.Get().GetOrElse(true)) != EOptionalFlag::Disabled;
 }
 
 EOptionalFlag TKikimrSettings::GetOptPredicateExtract() const {

Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,7 @@ bool TKikimrSettings::HasOptEnableOlapProvideComputeSharding() const {`
`145`	`145`	`}`
`146`	`146`
`147`	`147`	`bool TKikimrSettings::HasOptUseFinalizeByKey() const {`
`148`		`- return GetOptionalFlagValue(OptUseFinalizeByKey.Get()) != EOptionalFlag::Disabled;`
	`148`	`+ return GetFlagValue(OptUseFinalizeByKey.Get().GetOrElse(true)) != EOptionalFlag::Disabled;`
`149`	`149`	`}`
`150`	`150`
`151`	`151`	`EOptionalFlag TKikimrSettings::GetOptPredicateExtract() const {`