StarRocks / starrocks

The world's fastest open query engine for sub-second analytics both on and off the data lakehouse. With the flexibility to support nearly any scenario, StarRocks provides best-in-class performance for multi-dimensional analytics, real-time analytics, and ad-hoc queries. A Linux Foundation project.
https://starrocks.io
Apache License 2.0
9.03k stars 1.82k forks source link

[BugFix] Return error asap while executing ShortCircuitHybridScanNode. #53060

Open wuxueyang96 opened 1 day ago

wuxueyang96 commented 1 day ago

Why I'm doing:

When executing a point query through ShortCircuitHybridScanNode, if the tablet version is updated by another writing thread, it will fail because of a low read version, like below:

image

However, it will not throw any exceptions for users, and users may just get an empty result set.

What I'm doing:

Return error status as soon as possible while executing ShortCircuitHybridScanNode.

What type of PR is this:

Does this PR entail a change in behavior?

If yes, please specify the type of change:

Checklist:

Bugfix cherry-pick branch check:

github-actions[bot] commented 2 hours ago

[Java-Extensions Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] commented 2 hours ago

[FE Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] commented 1 hour ago

[BE Incremental Coverage Report]

:x: fail : 0 / 1 (00.00%)

file detail

path covered_line new_line coverage not_covered_line_detail
:large_blue_circle: be/src/exec/short_circuit_hybrid.cpp 0 1 00.00% [206]