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

   ### Rationale for this change
   
   One way of learning about a codebase is reading the tests. As it is now, 
it's hard to see the minimal `FlightServerBase` sub-class in 
`flight/test_util.cc`, so I moved it to its own file.
   
   ### What changes are included in this PR?
   
    - Renaming `FlightTestServer` to `TestFlightServer`
    - Moving the class to `test_flight_server.{h,cc}`
   
   ### Are these changes tested?
   
   By existing tests.
   
   ### Are there any user-facing changes?
   
   `ExampleTestServer` is removed from the testing library in favor of 
`FlightTestServer::Make`.


-- 
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: github-unsubscr...@arrow.apache.org

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

Reply via email to