Skip to content

[regression](case)fix mc regression test p2 case. - #42217

Merged
morningman merged 2 commits into
apache:masterfrom
hubgeter:fix_mc_regression_case
Oct 22, 2024
Merged

[regression](case)fix mc regression test p2 case.#42217
morningman merged 2 commits into
apache:masterfrom
hubgeter:fix_mc_regression_case

Conversation

@hubgeter

@hubgeterhubgeter commented Oct 21, 2024

Copy link
Copy Markdown
Contributor

Proposed changes

In previous versions, use a db that does not exist in show databases will not cause an error, so this test can be run before.

Since the test account does not have the list all projects permission, there is no db want to be used when show databases, which causes the case to fail.

Therefore, creat a new catalog and specify mc.default.project to fix this case.

@doris-robot

Copy link
Copy Markdown

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

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@hubgeter

Copy link
Copy Markdown
ContributorAuthor

run buildall

morningman
morningman previously approved these changes Oct 22, 2024

@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

@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actionsgithub-actionsBot added approved Indicates a PR has been approved by one committer. reviewed labels Oct 22, 2024
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@hubgeter

Copy link
Copy Markdown
ContributorAuthor

run buildall

@github-actionsgithub-actionsBot removed the approved Indicates a PR has been approved by one committer. label Oct 22, 2024
@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Oct 22, 2024
@github-actions

Copy link
Copy Markdown
Contributor

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

@morningman
morningman merged commit 7a89ec0 into apache:masterOct 22, 2024
morningman pushed a commit to morningman/doris that referenced this pull request Oct 22, 2024
## Proposed changes
In previous versions, use a db that does not exist in show databases
will not cause an error, so this test can be run before.
Since the test account does not have the list all projects permission,
there is no db want to be used when show databases, which causes the
case to fail.
Therefore, creat a new catalog and specify `mc.default.project` to fix
this case.
morningman pushed a commit to morningman/doris that referenced this pull request Oct 22, 2024
## Proposed changes
In previous versions, use a db that does not exist in show databases
will not cause an error, so this test can be run before.
Since the test account does not have the list all projects permission,
there is no db want to be used when show databases, which causes the
case to fail.
Therefore, creat a new catalog and specify `mc.default.project` to fix
this case.
morningman added a commit that referenced this pull request Oct 22, 2024
cherry pick from #42217
Co-authored-by: daidai <2017501503@qq.com>
morningman added a commit that referenced this pull request Oct 22, 2024
cherry pick from #42217
Co-authored-by: daidai <2017501503@qq.com>
HappenLee pushed a commit to HappenLee/incubator-doris that referenced this pull request Apr 24, 2026
## Proposed changes
In previous versions, use a db that does not exist in show databases
will not cause an error, so this test can be run before.
Since the test account does not have the list all projects permission,
there is no db want to be used when show databases, which causes the
case to fail.
Therefore, creat a new catalog and specify `mc.default.project` to fix
this case.
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.1.7-mergeddev/3.0.3-mergedreviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@hubgeter@doris-robot@morningman@CalvinKirs