[jira] [Assigned] (IMPALA-9339) Revise explanation of RowMaterializationTimer

["Kurt Deschler (Jira)" <ji...@apache.org>]

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

Kurt Deschler reassigned IMPALA-9339:
-------------------------------------

    Assignee: Pranav Yogi Lodha  (was: Sahil Takiar)

> Revise explanation of RowMaterializationTimer
> ---------------------------------------------
>
>                 Key: IMPALA-9339
>                 URL: https://issues.apache.org/jira/browse/IMPALA-9339
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Backend
>            Reporter: Sahil Takiar
>            Assignee: Pranav Yogi Lodha
>            Priority: Major
>
> IMPALA-8825 added the following explanation of the counter {{RowMaterializationTimer}}
> {quote}
> /// Tracks the time spent materializing rows and converting them into a QueryResultSet.
> /// The QueryResultSet format used depends on the client, for Beeswax clients an ASCII
> /// representation is used, whereas for HS2 clients (using TCLIService.thrift) rows are
> /// converted into a TRowSet. Materializing rows includes evaluating any yet unevaluated
> /// expressions using ScalarExprEvaluators.
> {quote}
> This isn't accurate because the actual timer measures time taken running {{Coordinator::GetNext}} which can block if any downstream operators in the plan take a long time to materialize rows.
> We should revise the explanation of {{RowMaterializationTimer}} and consider adding a separate timer that truly measures the amount of time taken to materialize rows (e.g. measure time taken in {{QueryResultSet::AddRows}}).



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

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-all-unsubscribe@impala.apache.org
For additional commands, e-mail: issues-all-help@impala.apache.org