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](regression) fix inverted_index_p1 q72.sql timeout error #19241

Merged
merged 2 commits into from
May 4, 2023

Conversation

ZhangYu0123
Copy link
Contributor

@ZhangYu0123 ZhangYu0123 commented May 3, 2023

Proposed changes

Fix inverted_index_p1 q72.sql timeout error
1、the runtime filter exeed wait time and lead to 100w * 1000w data join
2、the error:
image

Issue Number: close #xxx

Problem summary

Describe your changes.

Checklist(Required)

  • Does it affect the original behavior
  • Has unit tests been added
  • Has document been added or modified
  • Does it need to update dependencies
  • Is this PR support rollback (If NO, please explain WHY)

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

@ZhangYu0123
Copy link
Contributor Author

run buildall

@ZhangYu0123 ZhangYu0123 changed the title [fix](regression) test to fix p1 error [fix](regression) fix inverted_index_p1 q72.sql timeout error May 3, 2023
@ZhangYu0123
Copy link
Contributor Author

run buildall

@ZhangYu0123
Copy link
Contributor Author

run p0

@ZhangYu0123
Copy link
Contributor Author

run p1

@ZhangYu0123 ZhangYu0123 requested a review from yiguolei May 4, 2023 01:59
@ZhangYu0123 ZhangYu0123 marked this pull request as ready for review May 4, 2023 01:59
Copy link
Contributor

@yiguolei yiguolei left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label May 4, 2023
@github-actions
Copy link
Contributor

github-actions bot commented May 4, 2023

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

@github-actions
Copy link
Contributor

github-actions bot commented May 4, 2023

PR approved by anyone and no changes requested.

@yiguolei yiguolei merged commit aaf0ef7 into apache:master May 4, 2023
@ZhangYu0123 ZhangYu0123 deleted the p1_test_fix branch May 4, 2023 04:09
Reminiscent pushed a commit to Reminiscent/doris that referenced this pull request May 15, 2023
…#19241)

Fix inverted_index_p1 q72.sql timeout error
1、the runtime filter exeed wait time and lead to 100w * 1000w data join
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by one committer. kind/test reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants