BiteTheDDDDt commented on code in PR #52138:
URL: https://github.com/apache/doris/pull/52138#discussion_r2184859871
##########
be/src/vec/aggregate_functions/aggregate_function_null.h:
##########
@@ -39,6 +40,7 @@ class AggregateFunctionNullBaseInline : public
IAggregateFunctionHelper<Derived>
protected:
std::unique_ptr<NestFunction> nested_function;
size_t prefix_size;
+ mutable int64_t null_count = 0;
Review Comment:
这里的mutable的null_count看起来有点奇怪，为什么不是放在data里，是放在function里？是否需要添加一段注释来说明
--
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]
