This is an automated email from the ASF dual-hosted git repository.
lidavidm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new 8325ad1c16 GH-40422: [C++][FlightRPC] Add missing expiration_time
arguments (#40425)
8325ad1c16 is described below
commit 8325ad1c167faa74e70c9d43647fabdb2ac41b6a
Author: Hyunseok Seo <[email protected]>
AuthorDate: Sun Mar 10 02:27:39 2024 +0900
GH-40422: [C++][FlightRPC] Add missing expiration_time arguments (#40425)
### Rationale for this change
This happens when `-DARROW_BUILD_BENCHMARKS=ON` and sometimes cannot be
detected by CI.
### What changes are included in this PR?
Add missing `expiration_time` and `app_metadata` arguments as default.
### Are these changes tested?
yes (build and run unit test)
### Are there any user-facing changes?
No
* GitHub Issue: #40422
Authored-by: Hyunseok Seo <[email protected]>
Signed-off-by: David Li <[email protected]>
---
cpp/src/arrow/flight/integration_tests/test_integration.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cpp/src/arrow/flight/integration_tests/test_integration.cc
b/cpp/src/arrow/flight/integration_tests/test_integration.cc
index 464e6832c3..21f57efd12 100644
--- a/cpp/src/arrow/flight/integration_tests/test_integration.cc
+++ b/cpp/src/arrow/flight/integration_tests/test_integration.cc
@@ -2087,7 +2087,7 @@ class ReuseConnectionServer : public FlightServerBase {
const FlightDescriptor& descriptor,
std::unique_ptr<FlightInfo>* info) override {
auto location = Location::ReuseConnection();
- auto endpoint = FlightEndpoint{{"reuse"}, {location}};
+ auto endpoint = FlightEndpoint{{"reuse"}, {location}, std::nullopt, ""};
ARROW_ASSIGN_OR_RAISE(auto info_data, FlightInfo::Make(arrow::Schema({}),
descriptor,
{endpoint}, -1,
-1));
*info = std::make_unique<FlightInfo>(std::move(info_data));