Skip to content

[fix](regression) Add auto-expired prefix for backup/restore cases - #26272

Merged
morningman merged 1 commit into
apache:masterfrom
w41ter:case/add_backup_restore_prefix
Nov 2, 2023
Merged

[fix](regression) Add auto-expired prefix for backup/restore cases#26272
morningman merged 1 commit into
apache:masterfrom
w41ter:case/add_backup_restore_prefix

Conversation

@w41ter

Copy link
Copy Markdown
Contributor

Proposed changes

Issue Number: close #xxx

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@morningmanmorningman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@w41ter

Copy link
Copy Markdown
ContributorAuthor

run buildall

@dataroaringdataroaring left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Nov 2, 2023
@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@JackDrogonJackDrogon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@morningman
morningman merged commit 3e9e8be into apache:masterNov 2, 2023
@w41ter
w41ter deleted the case/add_backup_restore_prefix branch November 2, 2023 09:45
xiaokang pushed a commit that referenced this pull request Nov 3, 2023
* [feature](backup) add property to remove snapshot before creating repo (#25847)
Doris is not responsible for managing snapshots, but it needs to clear all
snapshots before doing backup/restore regression testing, so a property is
added to indicate that existing snapshots need to be cleared when creating a
repo.
In addition, a regression test case for backup/restore has been added.
* [cases](regression-test) Add backup & restore with multi tables test (#26040)
* [cases](regression-test) Add backup restore operation test (#26169)
1. restore overwrites an exists table
2. backup & restore with exclude table
3. restore to a new table
4. restore mix exists and new tables
5. restore with alias
* [fix](regression) Add auto-expired prefix for backup/restore cases (#26272)
* [test](regression) Add backup/restore DB case (#26312)
gnehil pushed a commit to gnehil/doris that referenced this pull request Dec 4, 2023
* [feature](backup) add property to remove snapshot before creating repo (apache#25847)
Doris is not responsible for managing snapshots, but it needs to clear all
snapshots before doing backup/restore regression testing, so a property is
added to indicate that existing snapshots need to be cleared when creating a
repo.
In addition, a regression test case for backup/restore has been added.
* [cases](regression-test) Add backup & restore with multi tables test (apache#26040)
* [cases](regression-test) Add backup restore operation test (apache#26169)
1. restore overwrites an exists table
2. backup & restore with exclude table
3. restore to a new table
4. restore mix exists and new tables
5. restore with alias
* [fix](regression) Add auto-expired prefix for backup/restore cases (apache#26272)
* [test](regression) Add backup/restore DB case (apache#26312)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.dev/2.0.3-mergedreviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@w41ter@morningman@JackDrogon@dataroaring