[ 
https://issues.apache.org/jira/browse/IMPALA-12163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17726400#comment-17726400
 ] 

ASF subversion and git services commented on IMPALA-12163:
----------------------------------------------------------

Commit ee300a1af09514af5650b0b14b90afc8d23d54b1 in impala's branch 
refs/heads/master from jasonmfehr
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=ee300a1af ]

IMPALA-12163: Fixes two issues when outputting RPC details.

The end time of the exact same rpc call was different between stdout
and the rpc details file because the end time was calculated each
time the details were written out instead of calculating the end time
once and reusing that value.

The duration of each rpc call was being calculated incorrectly.

Change-Id: Ifd9dec189d0f6fb8713fb1c7b2b6c663e492ef05
Reviewed-on: http://gerrit.cloudera.org:8080/19932
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>


> Shell RPC Duration Does Not Match Start/End Time
> ------------------------------------------------
>
>                 Key: IMPALA-12163
>                 URL: https://issues.apache.org/jira/browse/IMPALA-12163
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Clients
>            Reporter: Jason Fehr
>            Assignee: Jason Fehr
>            Priority: Minor
>              Labels: shell
>
> In the ImpalaHS2Client class within the impala_client.py file, the rpc 
> duration is calculated 
> [here|[https://github.com/apache/impala/blob/57964c83d37bdd61ec08b738c56725940138ab7d/shell/impala_client.py#LL1260-L1261].]
>   Then, the rpc end time is determined by calling datetime.now() 
> [here|[https://github.com/apache/impala/blob/57964c83d37bdd61ec08b738c56725940138ab7d/shell/impala_client.py#L1265].]
>   This leads to stdout and rpc file having different rpc end times.
> The duration is calculated incorrectly 
> [here|https://github.com/apache/impala/blob/57964c83d37bdd61ec08b738c56725940138ab7d/shell/impala_client.py#LL1267C22-L1267C22].
>   It needs to be calculated by calling the "total_seconds()" function.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to