-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix](arrow-flight-sql) Remove arrow_flight_sql group from regression test
#44489
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](arrow-flight-sql) Remove arrow_flight_sql group from regression test
#44489
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
arrow_flight_sql tag from regression testarrow_flight_sql group from regression test
61b6afc to
4c8e411
Compare
|
run buildall |
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
wangbo
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
…regression test (apache#44489)" This reverts commit 7a6bb51.
…on test (apache#44489) Problem Summary: Add `./run-regression-test.sh --run ... -g arrow_flight_sql` to the pipeline. then add the group `arrow_flight_sql` again, which is safer.
…on test (apache#44489) Problem Summary: Add `./run-regression-test.sh --run ... -g arrow_flight_sql` to the pipeline. then add the group `arrow_flight_sql` again, which is safer.
What problem does this PR solve?
Problem Summary:
Add
./run-regression-test.sh --run ... -g arrow_flight_sqlto the pipeline. then add the grouparrow_flight_sqlagain, which is safer.Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)