Skip to content

Commit 1103939

Browse files
authored
Re-enable c sharp arrow flight integration test (apache#6611)
1 parent 1295b00 commit 1103939

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

.github/workflows/integration.yml

+1-2
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,7 @@ jobs:
6060
ARROW_RUST_EXE_PATH: /build/rust/debug
6161
BUILD_DOCS_CPP: OFF
6262
ARROW_INTEGRATION_CPP: ON
63-
# Disable C# integration tests due to https://github.com/apache/arrow-rs/issues/6577
64-
ARROW_INTEGRATION_CSHARP: OFF
63+
ARROW_INTEGRATION_CSHARP: ON
6564
ARROW_INTEGRATION_GO: ON
6665
ARROW_INTEGRATION_JAVA: ON
6766
ARROW_INTEGRATION_JS: ON

arrow-integration-testing/src/flight_server_scenarios/integration_test.rs

+5-1
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,13 @@ type Result<T = (), E = Error> = std::result::Result<T, E>;
4848
/// Run a scenario that tests integration testing.
4949
pub async fn scenario_setup(port: u16) -> Result {
5050
let addr = super::listen_on(port).await?;
51+
let resolved_port = addr.port();
5152

5253
let service = FlightServiceImpl {
53-
server_location: format!("grpc+tcp://{addr}"),
54+
// See https://github.com/apache/arrow-rs/issues/6577
55+
// C# had trouble resolving addressed like 0.0.0.0:port
56+
// server_location: format!("grpc+tcp://{addr}"),
57+
server_location: format!("grpc+tcp://localhost:{resolved_port}"),
5458
..Default::default()
5559
};
5660
let svc = FlightServiceServer::new(service);

0 commit comments

Comments
 (0)