[ 
https://issues.apache.org/jira/browse/ARROW-7411?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Li resolved ARROW-7411.
-----------------------------
    Resolution: Fixed

Issue resolved by pull request 6051
[https://github.com/apache/arrow/pull/6051]

> [C++][Flight] Incorrect Arrow Flight benchmark output
> -----------------------------------------------------
>
>                 Key: ARROW-7411
>                 URL: https://issues.apache.org/jira/browse/ARROW-7411
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Benchmarking, C++, FlightRPC
>    Affects Versions: 0.15.1
>         Environment: macOS
>            Reporter: Chengxin Ma
>            Assignee: Chengxin Ma
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 1.0.0
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> When running Arrow Flight benchmark in the following scenario, the output is 
> incorrect. 
> {code}
> $ ./arrow-flight-perf-server &
> [1] 12986
> Server host: localhost
> Server port: 31337
> $ ./arrow-flight-benchmark -server_host localhost -test_put 
> Using remote server: true
> Testing method: DoPut
> Server host: localhost
> Server port: 31337
> Bytes read: 1280000000
> Nanos: 496372147
> Speed: 2459.25 MB/s
> {code}
> {{Using remote server}} should be {{false}} and {{Bytes read}} should be 
> {{Bytes write}}.
> To correct the result of {{Using remote server}}, we can:
> * Change {{if (FLAGS_server_host == "")}} to another condition which checks 
> if there is already an {{arrow-flight-perf-server}} running. This is a bit 
> complicated to do and might add some unnecessary complexity (e.g. we need to 
> make sure it support all OSes.). 
> * Delete {{Using remote server}}, since we already have {{Server host}} in 
> the output.
> I personally prefer the second option and will make a PR.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to