kou commented on code in PR #49171:
URL: https://github.com/apache/arrow/pull/49171#discussion_r2833507499
##########
dev/archery/archery/integration/runner.py:
##########
@@ -721,7 +721,7 @@ def run_all_tests(testers: List[Tester], other_testers:
List[Tester],
Scenario(
"flight_sql:extension",
description="Ensure Flight SQL extensions work as expected.",
- skip_testers={"Rust", ".NET"}
+ skip_testers={"Rust", ".NET", "C++"}
Review Comment:
> 1.Wait for those to be merged
Are there already PRs in apache/arrow-java and apache/arrow-go to migrate to
CancelFlightInfo from CancelQuery?
If you want to complete this PR as soon as possible, I prefer the following
step:
1. Remove CancelQuery related change from this PR
2. Merge this PR
3. Open a PR to apache/arrow-java to migrate to CancelFlightInfo from
CancelQuery
4. Merge the PR
5. Open a PR to apache/arrow-go to migrate to CancelFlightInfo from
CancelQuery
6. Merge the PR
7. Open a separated PR to this repository to migrate to CancelFlightInfo
from CancelQuery
8. Merge the PR
If you don't mind this PR keeps opening, I prefer the following step:
1. Open a PR to apache/arrow-java to migrate to CancelFlightInfo from
CancelQuery
2. Merge the PR
3. Open a PR to apache/arrow-go to migrate to CancelFlightInfo from
CancelQuery
4. Merge the PR
5. Remove the C++ skip from this PR
6. Merge this PR
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
