github-actions[bot] commented on code in PR #18324:
URL: https://github.com/apache/doris/pull/18324#discussion_r1155768368
##########
be/src/vec/exec/format/orc/vorc_reader.h:
##########
@@ -147,13 +147,19 @@ class OrcReader : public GenericReader {
Status _decode_explicit_decimal_column(const std::string& col_name,
const MutableColumnPtr& data_column,
const DataTypePtr& data_type,
- DecimalScaleParams& scale_params,
orc::ColumnVectorBatch* cvb, size_t
num_values) {
OrcColumnType* data = dynamic_cast<OrcColumnType*>(cvb);
if (data == nullptr) {
return Status::InternalError("Wrong data type for colum '{}'",
col_name);
}
- _init_decimal_converter<DecimalPrimitiveType>(data_type, scale_params,
data->scale);
+ if (_decimal_scale_params_index >= _decimal_scale_params.size()) {
+ DecimalScaleParams temp_scale_params;
+ _init_decimal_converter<DecimalPrimitiveType>(data_type,
temp_scale_params,
+ data->scale);
+ _decimal_scale_params.emplace_back(std::move(temp_scale_params));
Review Comment:
warning: std::move of the variable 'temp_scale_params' of the
trivially-copyable type 'doris::vectorized::DecimalScaleParams' has no effect;
remove std::move() [performance-move-const-arg]
```suggestion
_decimal_scale_params.emplace_back(temp_scale_params);
```
--
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
