Skip to content

[fix](fe) Fix NoSuchElementException when count with MATCH_ALL expression - #62172

Merged
BiteTheDDDDt merged 1 commit into
apache:masterfrom
HappenLee:function
Apr 8, 2026
Merged

[fix](fe) Fix NoSuchElementException when count with MATCH_ALL expression#62172
BiteTheDDDDt merged 1 commit into
apache:masterfrom
HappenLee:function

Conversation

@HappenLee

@HappenLeeHappenLee commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Problem Summary:
When executing explain on SQL like:
SELECT count(col MATCH_ALL 'xxx') FROM table WHERE ...
The Nereids optimizer throws NoSuchElementException because storageLayerAggregate rule tries to get originalColumn from a SlotReference that represents a virtual column (generated from MATCH_ALL expression). The virtual column has no physical column in the table schema, so getOriginalColumn() returns Optional.empty().

The fix adds proper isPresent() checks before calling get() on Optional.

Release note:
Fix a bug where count(MATCH_ALL) causes explain to fail with NoSuchElementException

Test:

  • Added regression test test_count_match_all_pushdown to cover this case

@HappenLee
[fix](fe) Fix NoSuchElementException when count with MATCH_ALL expres…
034800e
…sion
Issue Number: close #???
Problem Summary:
When executing explain on SQL like:
SELECT count(col MATCH_ALL 'xxx') FROM table WHERE ...
The Nereids optimizer throws NoSuchElementException because storageLayerAggregate
rule tries to get originalColumn from a SlotReference that represents a virtual
column (generated from MATCH_ALL expression). The virtual column has no physical
column in the table schema, so getOriginalColumn() returns Optional.empty().
The fix adds proper isPresent() checks before calling get() on Optional.
Release note:
Fix a bug where count(MATCH_ALL) causes explain to fail with NoSuchElementException
Test:
- Added regression test test_count_match_all_pushdown to cover this case
@Thearas

Thearas commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

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

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@HappenLee

HappenLee commented Apr 7, 2026

Copy link
Copy Markdown
ContributorAuthor

run buildall

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Apr 7, 2026
@github-actions

github-actionsBot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actions

github-actionsBot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@hello-stephen

hello-stephen commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

FE UT Coverage Report

Increment line coverage 25.00% (2/8) 🎉
Increment coverage report
Complete coverage report

@BiteTheDDDDt
BiteTheDDDDt merged commit c396335 into apache:masterApr 8, 2026
32 of 34 checks passed
github-actionsBot pushed a commit that referenced this pull request Apr 8, 2026
@HappenLeeYour Name
[fix](fe) Fix NoSuchElementException when count with MATCH_ALL expres…
7090a49
…sion (#62172)
When executing explain on SQL like:
SELECT count(col MATCH_ALL 'xxx') FROM table WHERE ...
The Nereids optimizer throws NoSuchElementException because
storageLayerAggregate rule tries to get originalColumn from a
SlotReference that represents a virtual column (generated from MATCH_ALL
expression). The virtual column has no physical column in the table
schema, so getOriginalColumn() returns Optional.empty().
The fix adds proper isPresent() checks before calling get() on Optional.
Release note:
Fix a bug where count(MATCH_ALL) causes explain to fail with
NoSuchElementException
Test:
- Added regression test test_count_match_all_pushdown to cover this case
@github-actionsgithub-actionsBot mentioned this pull request Apr 8, 2026
Closed
github-actionsBot pushed a commit that referenced this pull request Apr 8, 2026
@HappenLeeYour Name
[fix](fe) Fix NoSuchElementException when count with MATCH_ALL expres…
508f1ef
…sion (#62172)
When executing explain on SQL like:
SELECT count(col MATCH_ALL 'xxx') FROM table WHERE ...
The Nereids optimizer throws NoSuchElementException because
storageLayerAggregate rule tries to get originalColumn from a
SlotReference that represents a virtual column (generated from MATCH_ALL
expression). The virtual column has no physical column in the table
schema, so getOriginalColumn() returns Optional.empty().
The fix adds proper isPresent() checks before calling get() on Optional.
Release note:
Fix a bug where count(MATCH_ALL) causes explain to fail with
NoSuchElementException
Test:
- Added regression test test_count_match_all_pushdown to cover this case
@github-actionsgithub-actionsBot mentioned this pull request Apr 8, 2026
Merged
yiguolei pushed a commit that referenced this pull request Apr 8, 2026
@github-actions@HappenLee
branch-4.1: [fix](fe) Fix NoSuchElementException when count with MATC…
0e8e985
…H_ALL expression #62172 (#62198)
Cherry-picked from #62172
Co-authored-by: HappenLee <happenlee@selectdb.com>
HappenLee added a commit to HappenLee/incubator-doris that referenced this pull request Apr 15, 2026
@HappenLee
[fix](fe) Fix NoSuchElementException when count with MATCH_ALL expres…
ef3f458
…sion (apache#62172)
When executing explain on SQL like:
SELECT count(col MATCH_ALL 'xxx') FROM table WHERE ...
The Nereids optimizer throws NoSuchElementException because
storageLayerAggregate rule tries to get originalColumn from a
SlotReference that represents a virtual column (generated from MATCH_ALL
expression). The virtual column has no physical column in the table
schema, so getOriginalColumn() returns Optional.empty().
The fix adds proper isPresent() checks before calling get() on Optional.
Release note:
Fix a bug where count(MATCH_ALL) causes explain to fail with
NoSuchElementException
Test:
- Added regression test test_count_match_all_pushdown to cover this case
@mymeiyimymeiyi mentioned this pull request Apr 21, 2026
Merged
yiguolei pushed a commit that referenced this pull request May 1, 2026
@HappenLee
[fix](fe) Fix NoSuchElementException when count with MATCH_ALL expres…
ee0df3a
…sion (#62172) (#62518)
When executing explain on SQL like:
SELECT count(col MATCH_ALL 'xxx') FROM table WHERE ...
The Nereids optimizer throws NoSuchElementException because
storageLayerAggregate rule tries to get originalColumn from a
SlotReference that represents a virtual column (generated from MATCH_ALL
expression). The virtual column has no physical column in the table
schema, so getOriginalColumn() returns Optional.empty().
The fix adds proper isPresent() checks before calling get() on Optional.
Release note:
Fix a bug where count(MATCH_ALL) causes explain to fail with
NoSuchElementException
Test:
- Added regression test test_count_match_all_pushdown to cover this case
### What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
apache/doris-website#1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR
should merge into -->
@morningmanmorningman mentioned this pull request Jun 3, 2026
Closed
HappenLee added a commit that referenced this pull request Jun 24, 2026
@HappenLee
[fix](fe) Fix NoSuchElementException when count with MATCH_ALL expres…
5ad97f1
…sion (#62172)
When executing explain on SQL like:
SELECT count(col MATCH_ALL 'xxx') FROM table WHERE ...
The Nereids optimizer throws NoSuchElementException because
storageLayerAggregate rule tries to get originalColumn from a
SlotReference that represents a virtual column (generated from MATCH_ALL
expression). The virtual column has no physical column in the table
schema, so getOriginalColumn() returns Optional.empty().
The fix adds proper isPresent() checks before calling get() on Optional.
Release note:
Fix a bug where count(MATCH_ALL) causes explain to fail with
NoSuchElementException
Test:
- Added regression test test_count_match_all_pushdown to cover this case
@morningmanmorningman mentioned this pull request Jun 24, 2026
Merged
16 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BiteTheDDDDtBiteTheDDDDtBiteTheDDDDt approved these changes
@morrySnowmorrySnowmorrySnow approved these changes

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.dev/4.0.7-mergeddev/4.1.0-mergedreviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@HappenLee@Thearas@hello-stephen@BiteTheDDDDt@morrySnow@morningman@yiguolei