hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yin Huai (JIRA)" <>
Subject [jira] [Updated] (HIVE-4960) lastAlias in CommonJoinOperator is not used
Date Tue, 30 Jul 2013 21:19:48 GMT


Yin Huai updated HIVE-4960:

    Status: Patch Available  (was: Open)
> lastAlias in CommonJoinOperator is not used
> -------------------------------------------
>                 Key: HIVE-4960
>                 URL:
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Yin Huai
>            Assignee: Yin Huai
>            Priority: Minor
>         Attachments: HIVE-4960.D11895.1.patch
> In CommonJoinOperator, there is object called lastAlias. The initial value of this object
is 'null'. After tracing the usage of this object, I found that there is no place to change
the value of this object. Also, it is only used in processOp in JoinOperator and MapJoinOperator
> {code}
> if ((lastAlias == null) || (!lastAlias.equals(alias))) {
>   nextSz = joinEmitInterval;
> }
> {\code}
> Since lastAlias will always be null, we will assign joinEmitInterval to nextSz every
time we get a row. Later in processOp, we have 
> {code}
> nextSz = getNextSize(nextSz);
> {\code}
> Because we reset the value of nextSz to joinEmitInterval every time we get a row, seems
that getNextSize will not be used as expected.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message