fix

ydb-platform · Oct 24, 2024 · d913100 · d913100
1 parent 5f5c45b
commit d913100
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/ydb/core/protos/flat_scheme_op.proto b/ydb/core/protos/flat_scheme_op.proto
@@ -130,7 +130,7 @@ message TFamilyDescription {
     optional EColumnCache ColumnCache = 7;
     optional EColumnStorage Storage = 8; // DEPRECATED: use StorageConfig
     optional TStorageConfig StorageConfig = 9;
-    optional sint32 ColumnCodecLevel = 10;
+    optional int32 ColumnCodecLevel = 10;
 }
 
 message TFastSplitSettings {

diff --git a/ydb/library/yql/sql/v1/sql_translation.cpp b/ydb/library/yql/sql/v1/sql_translation.cpp
@@ -1572,7 +1572,11 @@ bool StoreString(const TRule_family_setting_value& from, TNodePtr& to, TContext&
         case TRule_family_setting_value::kAltFamilySettingValue1: {
             // STRING_VALUE
             const TString stringValue(ctx.Token(from.GetAlt_family_setting_value1().GetToken1()));
-            to = BuildLiteralSmartString(ctx, stringValue);
+            TNodePtr literal = BuildLiteralSmartString(ctx, stringValue);
+            if (literal->IsNull()) {
+                return false;
+            }
+            to = literal;
             break;
         }
         default:
@@ -1585,7 +1589,11 @@ bool StoreInt(const TRule_family_setting_value& from, TNodePtr& to, TContext& ct
     switch (from.Alt_case()) {
         case TRule_family_setting_value::kAltFamilySettingValue2: {
             // integer
-            to = LiteralNumber(ctx, from.GetAlt_family_setting_value2().GetRule_integer1());
+            TNodePtr literal = LiteralNumber(ctx, from.GetAlt_family_setting_value2().GetRule_integer1());
+            if (literal->IsNull()) {
+                return false;
+            }
+            to = literal;
             break;
         }
         default:

diff --git a/ydb/public/api/protos/ydb_table.proto b/ydb/public/api/protos/ydb_table.proto
@@ -536,7 +536,7 @@ message ColumnFamily {
 
     // Not all compression algorithms support
     // Set if want to change default value
-    optional sint32 compression_level = 5;
+    optional int32 compression_level = 5;
 }
 
 message PartitioningSettings {