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

Naveen Gangam commented on HIVE-9023:
-------------------------------------

In FileSinkOperator, I now used a [full] table-name based suffix for the 
Counter keys (similar to the earlier description) instead of just table IDs 
from the initial patch, for which HiveHistoryImpl looked up the name from the 
IdToTableMap (basically a map of integer ids-tablenames) generated by query 
plan. However, these table names were short table names without the database 
prefix.

So to get a full table name including the dbname, I am proposing the use of the 
value returned by the plan's Table descriptor API. 
getTableInfo().getTableName() as part of the Counter map's key (suffix) in the 
FileSinkOperator.

Does this help?

> HiveHistoryImpl relies on removed counters to print num rows
> ------------------------------------------------------------
>
>                 Key: HIVE-9023
>                 URL: https://issues.apache.org/jira/browse/HIVE-9023
>             Project: Hive
>          Issue Type: Bug
>          Components: Query Processor
>    Affects Versions: 0.13.0, 0.14.0, 0.13.1, 0.14.1
>            Reporter: Slava Markeyev
>            Assignee: Naveen Gangam
>            Priority: Minor
>              Labels: regression
>         Attachments: HIVE-9023.2.patch, HIVE-9023.patch
>
>
> HiveHistoryImpl still relies on the counters that were removed in HIVE-5982 
> to determine the number of rows loaded. This results in regression of 
> functionality. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to