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 c6e62f1fb5 MINOR: [C++][FlightRPC] Add missing app_metadata arguments 
(#38548)
c6e62f1fb5 is described below

commit c6e62f1fb54aecc1f240e7c8c3d2036279dab822
Author: Hyunseok Seo <[email protected]>
AuthorDate: Thu Nov 2 09:19:32 2023 +0900

    MINOR: [C++][FlightRPC] Add missing app_metadata arguments (#38548)
    
    
    
    ### Rationale for this change
    
    This issue was discovered in the #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]>
---
 cpp/src/arrow/flight/perf_server.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/arrow/flight/perf_server.cc 
b/cpp/src/arrow/flight/perf_server.cc
index 40f6cbcbf0..87676da112 100644
--- a/cpp/src/arrow/flight/perf_server.cc
+++ b/cpp/src/arrow/flight/perf_server.cc
@@ -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 =

Reply via email to