yiguolei commented on code in PR #24241:
URL: https://github.com/apache/doris/pull/24241#discussion_r1322758078


##########
be/src/vec/data_types/data_type_decimal.h:
##########
@@ -387,9 +387,9 @@ constexpr bool IsDataTypeDecimalOrNumber =
 
 template <typename FromDataType, typename ToDataType>
     requires IsDataTypeDecimal<FromDataType> && IsDataTypeDecimal<ToDataType>
-ToDataType::FieldType convert_decimals(const typename FromDataType::FieldType& 
value,
-                                       UInt32 scale_from, UInt32 scale_to,
-                                       UInt8* overflow_flag = nullptr) {
+typename ToDataType::FieldType convert_decimals(const typename 
FromDataType::FieldType& value,

Review Comment:
    why add this??



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to