Return default decimal if scheme proto is empty (#13555)

azevaykin · web-flow · commit 35b059a18221 · 2025-01-20T13:25:32.000+03:00
diff --git a/ydb/core/scheme/scheme_types_proto.cpp b/ydb/core/scheme/scheme_types_proto.cpp
@@ -92,8 +92,9 @@ NScheme::TTypeInfo TypeInfoFromProto(NScheme::TTypeId typeId, const ::NKikimrPro
         return NScheme::TTypeInfo(NPg::TypeDescFromPgTypeId(typeInfoProto.GetPgTypeId()));
     }
     case NScheme::NTypeIds::Decimal: {
-        Y_ABORT_UNLESS(typeInfoProto.HasDecimalPrecision());
-        Y_ABORT_UNLESS(typeInfoProto.HasDecimalScale());
+        if (!typeInfoProto.HasDecimalPrecision() || !typeInfoProto.HasDecimalScale()) {
+            return NScheme::TTypeInfo(NScheme::TDecimalType::Default());
+        }
         NScheme::TDecimalType decimal(typeInfoProto.GetDecimalPrecision(), typeInfoProto.GetDecimalScale());
         return NScheme::TTypeInfo(decimal);
     }

Original file line number	Diff line number	Diff line change
`@@ -92,8 +92,9 @@ NScheme::TTypeInfo TypeInfoFromProto(NScheme::TTypeId typeId, const ::NKikimrPro`
`92`	`92`	`return NScheme::TTypeInfo(NPg::TypeDescFromPgTypeId(typeInfoProto.GetPgTypeId()));`
`93`	`93`	`}`
`94`	`94`	`case NScheme::NTypeIds::Decimal: {`
`95`		`- Y_ABORT_UNLESS(typeInfoProto.HasDecimalPrecision());`
`96`		`- Y_ABORT_UNLESS(typeInfoProto.HasDecimalScale());`
	`95`	`+ if (!typeInfoProto.HasDecimalPrecision() \|\| !typeInfoProto.HasDecimalScale()) {`
	`96`	`+ return NScheme::TTypeInfo(NScheme::TDecimalType::Default());`
	`97`	`+ }`
`97`	`98`	`NScheme::TDecimalType decimal(typeInfoProto.GetDecimalPrecision(), typeInfoProto.GetDecimalScale());`
`98`	`99`	`return NScheme::TTypeInfo(decimal);`
`99`	`100`	`}`