[ https://issues.apache.org/jira/browse/HIVE-1642?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12932214#action_12932214 ]
Namit Jain commented on HIVE-1642: ---------------------------------- come more minor comments: 1203 //Qualify the path against the filesystem. The user configured path might co? ntain default port which is skipped 1204 //in the file status. This makes sure that all paths which goes into PathToP? artitionInfo are always listed status 1205 //filepath. 1206 newPath = fs.makeQualified(newPath); ExecDriver: is the above intentional ? MapJoinProcessor: public static String genMapJoinLocalWork(MapredWork newWork, MapJoinOperator mapJoinOp, Make it private -- add more comments Add more comments to the new functions // keep record all the input path for this alias HashSet<String> pathSet = new HashSet<String>(); for (Map.Entry<String, ArrayList<String>> entry2 : pathToAliases.entrySet()) { String path = entry2.getKey(); ArrayList<String> list = entry2.getValue(); if (list.contains(alias)) { // add to path set if (!pathSet.contains(path)) { pathSet.add(path); } // remove this alias from the alias list list.remove(alias); } } Don't you need to remove the entry from pathToAliases if list becomes empty. > Convert join queries to map-join based on size of table/row > ----------------------------------------------------------- > > Key: HIVE-1642 > URL: https://issues.apache.org/jira/browse/HIVE-1642 > Project: Hive > Issue Type: Improvement > Components: Query Processor > Reporter: Namit Jain > Assignee: Liyin Tang > Fix For: 0.7.0 > > Attachments: hive_1642_1.patch > > > Based on the number of rows and size of each table, Hive should automatically > be able to convert a join into map-join. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.