cast has been fixed for ArrowTypeAsYqlDatetime (#4491)

dorooleg · web-flow · commit 5cb09f9cef0b · 2024-05-14T23:01:06.000+03:00
diff --git a/ydb/library/yql/providers/s3/actors/yql_arrow_column_converters.cpp b/ydb/library/yql/providers/s3/actors/yql_arrow_column_converters.cpp
@@ -116,7 +116,7 @@ std::shared_ptr<arrow::Array> ArrowTypeAsYqlDatetime(const std::shared_ptr<arrow
             throw parquet::ParquetException(TStringBuilder() << "datetime in parquet is out of range [0, " << ::NYql::NUdf::MAX_DATETIME << "]: " << baseValue);
         }
 
-        const ui64 v = baseValue * multiplier;
+        const ui64 v = baseValue * static_cast<ui64>(multiplier);
         if (v > ::NYql::NUdf::MAX_DATETIME) {
             throw parquet::ParquetException(TStringBuilder() << "datetime in parquet is out of range [0, " << ::NYql::NUdf::MAX_DATETIME << "] after transformation: " << v);
         }

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,7 @@ std::shared_ptr<arrow::Array> ArrowTypeAsYqlDatetime(const std::shared_ptr<arrow`
`116`	`116`	`throw parquet::ParquetException(TStringBuilder() << "datetime in parquet is out of range [0, " << ::NYql::NUdf::MAX_DATETIME << "]: " << baseValue);`
`117`	`117`	`}`
`118`	`118`
`119`		`- const ui64 v = baseValue * multiplier;`
	`119`	`+ const ui64 v = baseValue * static_cast<ui64>(multiplier);`
`120`	`120`	`if (v > ::NYql::NUdf::MAX_DATETIME) {`
`121`	`121`	`throw parquet::ParquetException(TStringBuilder() << "datetime in parquet is out of range [0, " << ::NYql::NUdf::MAX_DATETIME << "] after transformation: " << v);`
`122`	`122`	`}`