raulcd commented on code in PR #49171:
URL: https://github.com/apache/arrow/pull/49171#discussion_r2833471938
##########
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:
We should create the PRs to remove CancelQuery on Arrow Java and Arrow Go
and hold this PR until this is done. Skipping the whole flight sql integration
tests could introduce integration issues without us noticing. You could also
split the deprecation of CancelQuery to a follow up PR until the other PRs are
merged (possibly add a TODO explaining why it can't be deprecated yet) and just
push the other deprecation removals. Both approaches seem reasonable.
--
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]
