airborne12 opened a new pull request, #69011:
URL: https://github.com/apache/doris/pull/69011
### What problem does this PR solve?
Issue Number: N/A
Related PR: N/A
Problem Summary:
The multi-analyzer inverted-index regression waits for the latest
schema-change and index-build jobs by table name. Doris retains completed jobs
after a table is dropped and recreated with the same name. A local check found
FINISHED jobs for this test table while the table did not exist, so the test
can mistake a historical job for the work it just requested.
Cause:
The waits do not identify jobs created by their own ALTER TABLE and BUILD
INDEX statements. They also wait until timeout when a job enters a failed
terminal state.
Changes:
- Record the highest JobId before each asynchronous statement and wait only
for newer jobs.
- Require all new jobs to finish, fail immediately on CANCELLED or FAILED,
and include job details in failure diagnostics.
- Correct the cloud BUILD INDEX comment.
Tests:
- The focused non-cloud regression passed twice consecutively with
`./run-regression-test.sh --run -s test_multi_tokenize_index_not_built -d
inverted_index_p0 -times 2`. All six query results matched the golden file.
- A local SQL check confirmed that completed schema-change and build-index
jobs remain visible after the table is dropped.
- Cloud mode was not run locally because no cloud cluster is available. The
exact failure stage in the original CI runs remains unconfirmed because their
logs were unavailable without credentials.
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [x] 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:
- [x] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [x] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/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 -->
--
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]
