Skip to content

[test](mtmv) Fix regression test unstable - #40871

Merged
morrySnow merged 4 commits into
apache:masterfrom
seawinde:fix_branch_unstable
Sep 20, 2024
Merged

[test](mtmv) Fix regression test unstable#40871
morrySnow merged 4 commits into
apache:masterfrom
seawinde:fix_branch_unstable

Conversation

@seawinde

@seawindeseawinde commented Sep 14, 2024

Copy link
Copy Markdown
Member

Proposed changes

  1. Optimized waitForRollUpJob method in Suite.groovy to make sure roll up is build succesfully before rewirte.
  2. Modify expect rewritten result in agg_sync_mv.groovy

@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.

@seawinde

Copy link
Copy Markdown
MemberAuthor

run buildall

(3, 1, 'o', 33.5, '2023-12-10', 'a', 'b', 1, 'yy'),
(3, 1, 'o', 33.5, '2023-12-10', 'a', 'b', 1, 'yy'),
(3, 1, 'o', 33.5, '2023-12-10', 'a', 'b', 1, 'yy'),
(4, 2, 'o', 43.2, '2023-12-11', 'c','d',2, 'mm'),

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.

why update data?

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Becaus the case is not chosen by CBO stably, add more data to make cbo chose mv easier.
but there is a fix pr #40529 to fix this. Maybe we doesn't need to update data.

@seawinde

Copy link
Copy Markdown
MemberAuthor

run buildall

@seawinde

Copy link
Copy Markdown
MemberAuthor

run buildall

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Sep 19, 2024
@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.

@morrySnow
morrySnow merged commit da6ac0c into apache:masterSep 20, 2024
seawinde added a commit to seawinde/doris that referenced this pull request Sep 20, 2024
1. Optimized `waitForRollUpJob` method in `Suite.groovy` to make sure
roll up is build succesfully before rewirte.
2. Modify expect rewritten result in` agg_sync_mv.groovy`
yiguolei pushed a commit that referenced this pull request Sep 20, 2024
seawinde added a commit to seawinde/doris that referenced this pull request Oct 23, 2024
1. Optimized `waitForRollUpJob` method in `Suite.groovy` to make sure
roll up is build succesfully before rewirte.
2. Modify expect rewritten result in` agg_sync_mv.groovy`
dataroaring pushed a commit that referenced this pull request Oct 23, 2024
HappenLee pushed a commit to HappenLee/incubator-doris that referenced this pull request Apr 24, 2026
1. Optimized `waitForRollUpJob` method in `Suite.groovy` to make sure
roll up is build succesfully before rewirte.
2. Modify expect rewritten result in` agg_sync_mv.groovy`
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.

6 participants

@seawinde@doris-robot@starocean999@morrySnow@yiguolei@dataroaring