alamb opened a new pull request #9593:
URL: https://github.com/apache/arrow/pull/9593


   # Background
   Thanks to the 🦅 👁️  of @lidavidm 
https://github.com/apache/arrow/pull/9583#issuecomment-786925049 it appears 
that the Rust flight integration test prints `"Server listening"` before the 
server is *actually* read to accept connections. Thus creating a race 
condition: if the tests try and start faster than the rust server can actually 
be setup then we see integration test failures as shown in 
https://issues.apache.org/jira/browse/ARROW-11717:
   
   ```
   Traceback (most recent call last):
     File "/arrow/dev/archery/archery/integration/runner.py", line 308, in 
_run_flight_test_case
       consumer.flight_request(port, **client_args)
     File "/arrow/dev/archery/archery/integration/tester_cpp.py", line 116, in 
flight_request
       run_cmd(cmd)
     File "/arrow/dev/archery/archery/integration/util.py", line 148, in run_cmd
       raise RuntimeError(sio.getvalue())
   RuntimeError: Command failed: 
/build/cpp/debug/flight-test-integration-client -host localhost -port=33569 
-scenario auth:basic_proto
   With output:
   --------------
   -- Arrow Fatal Error --
   Invalid: Expected UNAUTHENTICATED but got Unavailable
   ```
   
   # Changes
   This PR changes the Rust flight scenario harnesses to print the ready 
message after the server is *actually* ready.
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to