Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix](hint) fix hint tests with different be instances #35659

Merged
merged 1 commit into from
May 30, 2024

Conversation

LiBinfeng-01
Copy link
Contributor

cherry-pick: #35188
Problem:
When using multiple be to test hint with distribute hint, the result would be unstable Solved:
Add ordered hint to every distribute hint and move some leading hint cases to check containing of hint infomation

Proposed changes

Issue Number: close #xxx

Problem:
When using multiple be to test hint with distribute hint, the result would be unstable
Solved:
Add ordered hint to every distribute hint and move some leading hint cases to check containing of hint infomation
@doris-robot
Copy link

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.

@LiBinfeng-01
Copy link
Contributor Author

run buildall

@morrySnow morrySnow changed the title [fix](hint) fix hint tests with different be instances (#35188) [fix](hint) fix hint tests with different be instances May 30, 2024
@morrySnow
Copy link
Contributor

run p0

@xiaokang xiaokang merged commit c7ab41a into apache:branch-2.0 May 30, 2024
25 of 26 checks passed
mongo360 pushed a commit to mongo360/doris that referenced this pull request Aug 16, 2024
…pache#35659)

Problem:
When using multiple be to test hint with distribute hint, the result
would be unstable Solved:
Add ordered hint to every distribute hint and move some leading hint
cases to check containing of hint infomation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants