HappenLee commented on PR #67965: URL: https://github.com/apache/doris/pull/67965#issuecomment-5727795548
建议补充性能测试数据后再完整评估这个改动的性能影响。 `CastToDecimal::_from_float` 是逐值执行的热路径，本 PR 在原有乘法/转换基础上新增了 `std::round`、原始整数部分检查以及转换后的边界检查；Decimal256 还涉及更宽整数类型的转换。当前 PR 描述和测试结果只覆盖正确性，没有给出性能基线。 建议至少提供： - 修改前后的吞吐或单行耗时，以及相对变化百分比； - DOUBLE/FLOAT 分别转换到 DECIMAL64、DECIMAL128、DECIMAL256； - scale=0、普通 scale 和高 scale； - strict/non-strict 两种模式； - 足够大的数据量，并说明是否包含溢出和 NULL 数据； - 相同机器、编译类型和测试命令，最好给出多次运行的平均值或 P50/P99。 如果性能变化低于噪声范围，也请给出实际测量数据，而不是只说明“没有明显影响”。 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
