[jira] [Updated] (ARROW-7411) [C++][Flight] Incorrect Arrow Flight benchmark output

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

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

ASF GitHub Bot updated ARROW-7411:
----------------------------------
    Labels: pull-request-available  (was: )

> [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
>
>
> 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)