-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix](regression-test) use correct dataset for unique_with_mow_p2 #25653
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-test) use correct dataset for unique_with_mow_p2 #25653
Conversation
|
run buildall |
df565f1 to
d734cef
Compare
bde9fc5 to
f960c86
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
2 similar comments
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
342ad75 to
f960c86
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
1baf769 to
df6b32c
Compare
|
run buildall |
liaoxin01
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
PR approved by anyone and no changes requested. |
dataroaring
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
PR approved by at least one committer and no changes requested. |
Proposed changes
Issue Number: close #xxx
cases in
unique_with_mow_p2should use sf100 dataset, rather than sf0.1 datasetsf100 dataset have 10 data files for lineorder table, which is not easy to load through stream load, so I use s3 load instead.
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...