Skip to content

Conversation

@airborne12
Copy link
Member

cherry pick from #40953

…che#40953)

fix problem like this:
```
set enable_common_expr_pushdown=false;

create table t(a int, c varchar(1024)) PROPERTIES ("replication_allocation" = "tag.location.default: 1");

insert into t values(1, "a b c d e");

SELECT count() FROM t where c MATCH_PHRASE_EDGE 's';

I20240919 09:31:40.713896 2224044 internal_service.cpp:626] Cancel query 3e48a306704f4691-840d239bf31420aa, reason: [INTERNAL_ERROR]PStatus: (127.0.0.1)[INTERNAL_ERROR]Column match_phrase_edge({"c", "String"}) in block is nullptr, in method bytes. All Columns are c String match_phrase_edge({"c", "String"})
        3#  doris::vectorized::Block::allocated_bytes() const at be/src/vec/core/block.cpp:446
        4#  doris::vectorized::ScannerScheduler::_scanner_scan(std::shared_ptr<doris::vectorized::ScannerContext>, std::shared_ptr<doris::vectorized::ScanTask>) at be/src/vec/exec/scan/scanner_scheduler.cpp:243
```
@airborne12
Copy link
Member Author

run buildall

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 36.16% (9328/25800)
Line Coverage: 27.72% (76638/276517)
Region Coverage: 26.49% (39355/148539)
Branch Coverage: 23.30% (20042/86032)
Coverage Report: http://coverage.selectdb-in.cc/coverage/f3d6b4554efe86ae315f86208eb0a525d1c4e166_f3d6b4554efe86ae315f86208eb0a525d1c4e166/report/index.html

@airborne12 airborne12 merged commit 68b7783 into apache:branch-2.1 Sep 20, 2024
@airborne12 airborne12 deleted the pick_40953_to_origin_branch-2.1 branch September 20, 2024 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants