rtadepalli opened a new pull request, #34357:
URL: https://github.com/apache/arrow/pull/34357

   ### Rationale for this change
   
   Today, the `FlightServer` is created in `FlightTestUtil#getStartedServer`. 
All tests that need a server go through this function, but this is unnecessary 
since the server could directly be started on port 0 (the OS will assign a 
random port later).
   
   ### What changes are included in this PR?
   
   All invocations of `FlightTestUtil#getStartedServer` have been removed, and 
the burden of generating a `Location` object for the server has been placed on 
the caller.
   
   ### Are these changes tested?
   
   Existing tests impacted were run locally to make sure that there no 
regressions.
   
   ### Are there any user-facing changes?
   
   There are no user facing changes. There are no breaking changes to the 
public API as well.
   
   @lidavidm


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

Reply via email to