HappenLee commented on a change in pull request #4152:
URL: https://github.com/apache/incubator-doris/pull/4152#discussion_r459824447
##########
File path: be/src/runtime/buffered_tuple_stream2.h
##########
@@ -257,6 +257,11 @@ class BufferedTupleStream2 {
bool has_tuple_footprint() const {
return _fixed_tuple_row_size > 0 || !_string_slots.empty() ||
_nullable_tuple;
}
+ /// Returns true if the row consumes any memory. If false, the stream only
needs to
+ /// store the count of rows.
+ bool row_consumes_memory() const {
Review comment:
I will change the fun name
----------------------------------------------------------------
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.
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
