[
https://issues.apache.org/jira/browse/IMPALA-9339?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Tim Armstrong updated IMPALA-9339:
----------------------------------
Component/s: Backend
> 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: Sahil Takiar
> 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.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
