Skip to content

[fix](regression) Run restore auth case non-concurrently on branch-4.1 (#65668) - #67620

Open
gavinchou wants to merge 1 commit into
apache:branch-4.1from
gavinchou:gavin-pick-65668-branch-4.1
Open

[fix](regression) Run restore auth case non-concurrently on branch-4.1 (#65668)#67620
gavinchou wants to merge 1 commit into
apache:branch-4.1from
gavinchou:gavin-pick-65668-branch-4.1

Conversation

@gavinchou

Copy link
Copy Markdown
Contributor

What problem does this PR solve?

Related PR: #65668

Backport #65668 to branch-4.1. When test_ddl_restore_auth submits a backup concurrently with the repository authorization case, it can hold BackupHandler's global submission lock beyond the latter's 10-second DROP REPOSITORY lock timeout.

Add test_ddl_restore_auth to the nonConcurrent group, consistent with test_ddl_backup_auth. The original commit is e7b0e4e22d538447f3bd31ecf89c5d2cf5b16665. The cherry-pick required resolving context around the older suite declaration; the final change is the same one-line group addition.

Release note

None

Check List (For Author)

  • Test:
    • Manual test: static validation with git diff --check and review of the branch-4.1 scheduler. It runs nonConcurrent suites through its single-thread executor after ordinary suites complete.
    • Regression test: not run locally; this workspace has no running Doris test cluster.
  • Behavior changed:
    • Yes: this regression suite runs in the serial phase.
  • Does this need documentation?
    • No.

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

apache#65668)

### What problem does this PR solve?

Related PR: apache#65668

Problem Summary:

Backport the master fix to branch-4.1. A concurrent backup submission in
test_ddl_restore_auth can hold BackupHandler's global submission lock
long enough for test_ddl_repository_auth to exceed the 10-second lock
wait when dropping its repository.

Add test_ddl_restore_auth to the nonConcurrent group, consistent with
test_ddl_backup_auth. Resolve the cherry-pick context conflict against
the older branch-4.1 suite declaration while retaining its test body.

### Release note

None

### Check List (For Author)

- Test: git diff --check and static review of the branch-4.1 scheduler.
  The regression suite was not run locally; this workspace has no running
  Doris test cluster.
- Behavior changed: Yes, the regression suite runs in the serial phase.
- Does this need documentation: No.

(cherry picked from commit e7b0e4e)
@gavinchou
gavinchou requested a review from yiguolei as a code owner September 7, 2026 12:27
@hello-stephen

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?

@hello-stephen

Copy link
Copy Markdown
Contributor

skip buildall

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.

3 participants