[
https://issues.apache.org/jira/browse/IMPALA-12163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jason Fehr updated IMPALA-12163:
--------------------------------
Description:
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.
was:
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 the rpc end time not matching the duration. It also leads to
stdout and rpc file having different rpc end times.
Fix this issue by using the variable "end_time" instead of calling
datetime.now() multiple times.
> 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: issues-all-unsubscr...@impala.apache.org
For additional commands, e-mail: issues-all-h...@impala.apache.org
