AliRana30 commented on PR #49171: URL: https://github.com/apache/arrow/pull/49171#issuecomment-3933884773
@kou You're absolutely right - skipping C++ would skip all extension tests, not just CancelQuery. Looking at the code, the C++ integration test already uses [CancelFlightInfo] (the replacement API). The failures were happening because the integration test servers in Java/Go still expect the CancelQuery action. Would you prefer: **1.**I remove CancelQuery from Java/Go integration test servers as well (they also deprecated it), OR **2.**I keep the C++ skip for now, and we can remove CancelQuery from all languages in a coordinated PR? Option 1 would make this PR more complete, but touches Java/Go code beyond just C++. -- 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]
