Skip to content

[fix](regression) isolate ann_index_p0 table names#62178

Merged
airborne12 merged 2 commits intoapache:masterfrom
zhiqiang-hhhh:fix-ann-index-p0-table-name-isolation
Apr 8, 2026
Merged

[fix](regression) isolate ann_index_p0 table names#62178
airborne12 merged 2 commits intoapache:masterfrom
zhiqiang-hhhh:fix-ann-index-p0-table-name-isolation

Conversation

@zhiqiang-hhhh
Copy link
Copy Markdown
Contributor

Summary

  • rename shared ANN regression tables in ann_index_basic to suite-unique names
  • rename shared ANN regression tables in ivf_index_test to suite-unique names
  • avoid cross-suite table collisions in ann_index_p0 when suites run in parallel against the same regression database

@Thearas
Copy link
Copy Markdown
Contributor

Thearas commented Apr 7, 2026

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?

@zhiqiang-hhhh zhiqiang-hhhh force-pushed the fix-ann-index-p0-table-name-isolation branch from 3d257a3 to 9c08781 Compare April 7, 2026 13:53
@zhiqiang-hhhh
Copy link
Copy Markdown
Contributor Author

run buildall

### What problem does this PR solve?

Issue Number: None

Related PR: None

Problem Summary: IVF and IVF_ON_DISK regression cases only executed several ANN queries without asserting their results, and some snapshot queries relied on implicit row order. This could miss regressions in ANN behavior and make the cases flaky.

### Release note

None

### Check List (For Author)

- Test: Regression test
    - `./run-regression-test.sh --run -f regression-test/suites/ann_index_p0/ivf_index_test.groovy -forceGenOut`
    - `./run-regression-test.sh --run -f regression-test/suites/ann_index_p0/ivf_on_disk_index_test.groovy -forceGenOut`
- Behavior changed: No
- Does this need documentation: No
@zhiqiang-hhhh
Copy link
Copy Markdown
Contributor Author

run buildall

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

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

@github-actions github-actions bot added approved Indicates a PR has been approved by one committer. reviewed labels Apr 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

PR approved by anyone and no changes requested.

Copy link
Copy Markdown
Member

@airborne12 airborne12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@airborne12 airborne12 merged commit 9c226f5 into apache:master Apr 8, 2026
30 of 31 checks passed
zhiqiang-hhhh added a commit to zhiqiang-hhhh/doris that referenced this pull request Apr 8, 2026
yiguolei pushed a commit that referenced this pull request Apr 9, 2026
…es (#62215)

## Summary
- backport PR #60358, #61160 and #62178 into branch-4.1 as a single
commit
- add IVF on-disk ANN index support, related cache/runtime changes, and
FE session/property updates
- bring over ANN regression coverage updates for IVF, IVF on-disk,
small-segment and min-train-rows scenarios
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. dev/4.1.0-merged reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants