Skip to content

Commit

Permalink
MINOR: [C++][FlightRPC] Add missing app_metadata arguments (apache#38548
Browse files Browse the repository at this point in the history
)

### Rationale for this change

This issue was discovered in the apache#38210 issue. It is reproduced on Mac with ARM Architecture CPU (**M1**). If the argument for `app_metadata` is not explicitly added, the build fails.

### What changes are included in this PR?

* Added the argument for `app_metadata` explicitly

### Are these changes tested?

Yes (Build)

### Are there any user-facing changes?

No

Authored-by: Hyunseok Seo <[email protected]>
Signed-off-by: David Li <[email protected]>
  • Loading branch information
llama90 authored and loicalleyne committed Nov 13, 2023
1 parent 6d22eb7 commit 2f236fb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cpp/src/arrow/flight/perf_server.cc
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@ class FlightPerfServer : public FlightServerBase {
(void)token.SerializeToString(&tmp_ticket.ticket);

// All endpoints same location for now
endpoints.push_back(FlightEndpoint{tmp_ticket, {location_}, std::nullopt});
endpoints.push_back(FlightEndpoint{tmp_ticket, {location_}, std::nullopt, ""});
}

uint64_t total_records =
Expand Down

0 comments on commit 2f236fb

Please sign in to comment.