yiguolei commented on code in PR #60367:
URL: https://github.com/apache/doris/pull/60367#discussion_r2772182880
##########
be/src/vec/common/hash_table/join_hash_table.h:
##########
@@ -35,6 +36,29 @@ inline uint32_t hash_join_table_calc_bucket_size(size_t
num_elem) {
static_cast<size_t>(std::numeric_limits<int32_t>::max()) + 1);
}
+// Estimate the memory size needed for hash table basic structures (first,
next, visited).
+// When include_key_storage is true, also estimates memory for hash map key
storage
+// (stored_keys and bucket_nums), which provides a rough approximation without
+// knowing the exact hash map context type.
+inline size_t estimate_hash_table_mem_size(size_t rows, TJoinOp::type join_op,
+ bool include_key_storage = false) {
Review Comment:
为什么这里还有一个默认值？
--
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]
