This is an automated email from the ASF dual-hosted git repository.
alamb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git
The following commit(s) were added to refs/heads/master by this push:
new 110393920c Re-enable c sharp arrow flight integration test (#6611)
110393920c is described below
commit 110393920cf504b29afb802e8d574efa77418ad4
Author: Andrew Lamb <[email protected]>
AuthorDate: Thu Oct 24 09:57:19 2024 -0400
Re-enable c sharp arrow flight integration test (#6611)
---
.github/workflows/integration.yml | 3 +--
.../src/flight_server_scenarios/integration_test.rs | 6 +++++-
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/.github/workflows/integration.yml
b/.github/workflows/integration.yml
index 4e91844ef6..868729a168 100644
--- a/.github/workflows/integration.yml
+++ b/.github/workflows/integration.yml
@@ -60,8 +60,7 @@ jobs:
ARROW_RUST_EXE_PATH: /build/rust/debug
BUILD_DOCS_CPP: OFF
ARROW_INTEGRATION_CPP: ON
- # Disable C# integration tests due to
https://github.com/apache/arrow-rs/issues/6577
- ARROW_INTEGRATION_CSHARP: OFF
+ ARROW_INTEGRATION_CSHARP: ON
ARROW_INTEGRATION_GO: ON
ARROW_INTEGRATION_JAVA: ON
ARROW_INTEGRATION_JS: ON
diff --git
a/arrow-integration-testing/src/flight_server_scenarios/integration_test.rs
b/arrow-integration-testing/src/flight_server_scenarios/integration_test.rs
index 0c58fae93d..0f404b2ae2 100644
--- a/arrow-integration-testing/src/flight_server_scenarios/integration_test.rs
+++ b/arrow-integration-testing/src/flight_server_scenarios/integration_test.rs
@@ -48,9 +48,13 @@ type Result<T = (), E = Error> = std::result::Result<T, E>;
/// Run a scenario that tests integration testing.
pub async fn scenario_setup(port: u16) -> Result {
let addr = super::listen_on(port).await?;
+ let resolved_port = addr.port();
let service = FlightServiceImpl {
- server_location: format!("grpc+tcp://{addr}"),
+ // See https://github.com/apache/arrow-rs/issues/6577
+ // C# had trouble resolving addressed like 0.0.0.0:port
+ // server_location: format!("grpc+tcp://{addr}"),
+ server_location: format!("grpc+tcp://localhost:{resolved_port}"),
..Default::default()
};
let svc = FlightServiceServer::new(service);