kou commented on code in PR #40084:
URL: https://github.com/apache/arrow/pull/40084#discussion_r1511985858
##########
cpp/src/arrow/flight/types.h:
##########
@@ -424,6 +424,14 @@ struct ARROW_FLIGHT_EXPORT Location {
/// \brief Initialize a location by parsing a URI string
static arrow::Result<Location> Parse(const std::string& uri_string);
+ /// \brief Get the fallback URI.
+ ///
+ /// arrow-flight-reuse-connection:// means that a client may attempt to
Review Comment:
```suggestion
/// arrow-flight-reuse-connection://? means that a client may attempt to
```
##########
cpp/src/arrow/flight/integration_tests/test_integration.cc:
##########
@@ -2079,6 +2079,50 @@ class FlightSqlExtensionScenario : public
FlightSqlScenario {
return Status::OK();
}
};
+
+/// \brief The server for testing arrow-flight-reuse-connection://.
+class ReuseConnectionServer : public FlightServerBase {
+ public:
+ Status GetFlightInfo(const ServerCallContext& context,
+ const FlightDescriptor& descriptor,
+ std::unique_ptr<FlightInfo>* info) override {
+ auto location = Location::ReuseConnection();
+ auto endpoint = FlightEndpoint{{"reuse"}, {location}};
+ ARROW_ASSIGN_OR_RAISE(auto info_data, FlightInfo::Make(arrow::Schema({}),
descriptor,
+ {endpoint}, -1,
-1));
+ *info = std::make_unique<FlightInfo>(std::move(info_data));
+ return Status::OK();
+ }
+};
+
+/// \brief A scenario for testing arrow-flight-reuse-connection://.
Review Comment:
```suggestion
/// \brief A scenario for testing arrow-flight-reuse-connection://?.
```
--
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]
