wumeibanfa commented on code in PR #55719:
URL: https://github.com/apache/doris/pull/55719#discussion_r2343085129
##########
be/src/exec/rowid_fetcher.cpp:
##########
@@ -900,6 +900,8 @@ Status RowIdStorageReader::read_batch_external_row(
// Insert the read data into result_block.
for (size_t column_id = 0; column_id < result_block.get_columns().size();
column_id++) {
+ // The non-const Block(result_block) is passed in read_by_rowids, but
columns[i] in get_columns
Review Comment:
这个我认为下游也应该修改，下游使用了assume_mutate，但是这样也许得修改一整条调用链，或许改动太大了？
--
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]
