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

Namit Jain commented on HIVE-1348:
----------------------------------

The fix for https://issues.apache.org/jira/browse/HIVE-1388 can be better 
insulated at CombineFileRecordReader.
So, there will not be any conflicts with this - will commit this one right now.

+1

> Moving inputFileChanged() from ExecMapper to where it is needed
> ---------------------------------------------------------------
>
>                 Key: HIVE-1348
>                 URL: https://issues.apache.org/jira/browse/HIVE-1348
>             Project: Hadoop Hive
>          Issue Type: Improvement
>            Reporter: Ning Zhang
>            Assignee: He Yongqiang
>         Attachments: hive-1348.1.patch, hive-1348.2.patch, hive-1348.3.patch, 
> HIVE-1348.4.patch, HIVE-1348.5.patch
>
>
> inputFileChanged() is only needed for Bucketed sort merge map join. It should 
> not be put in ExecMapper.map() where all code paths will hit this function. 
> This function is quite expensive since JobConf look up is a hash table look 
> up. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to