Copilot commented on code in PR #64641:
URL: https://github.com/apache/doris/pull/64641#discussion_r3435268078
##########
be/src/exec/common/variant_util.cpp:
##########
@@ -1010,6 +1014,10 @@ void VariantCompactionUtil::get_subpaths(int32_t
max_subcolumns_count,
} else {
// Apply all paths as subcolumns
for (const auto& [path, _] : stats) {
Review Comment:
The comment here is no longer accurate after routing empty paths to
`sparse_path_set`: this branch no longer applies *all* paths as subcolumns (it
now keeps the empty/root path sparse). Updating the comment will help future
readers understand why `""` is excluded from subcolumn materialization.
--
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]
