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

Namit Jain commented on HIVE-417:
---------------------------------

        if (work.getReducer() != null) {
          work.getReducer().jobClose(job, success, feedBack);
        }
        
        if (IndexBuilderBaseReducer.class.isAssignableFrom(this
            .getReducerClass())) {
          this.closeIndexBuilder(job, success);
        }
      }


Instead of the above code in ExecDriver,  
IndexBuilderBaseReducer/CompactSumReducer should have a jobClose - no code
change needed in ExecDriver.

I would still vote for the index code to be in contrib, it will take some time 
to clean it up - then it should be moved to the mainline.
Till then, it is usable, but in a prototype state.

What we should aim for is minimum changes in ql/. and put all changes in 
contrib for now. As they become stable, we can pull them
in - even the DDLSemanticAnalyzer should be factored in contrib

> Implement Indexing in Hive
> --------------------------
>
>                 Key: HIVE-417
>                 URL: https://issues.apache.org/jira/browse/HIVE-417
>             Project: Hadoop Hive
>          Issue Type: New Feature
>          Components: Metastore, Query Processor
>    Affects Versions: 0.3.0, 0.3.1, 0.4.0, 0.6.0
>            Reporter: Prasad Chakka
>            Assignee: He Yongqiang
>         Attachments: hive-417.proto.patch, hive-417-2009-07-18.patch, 
> hive-indexing.3.patch, hive-indexing.5.thrift.patch, 
> indexing_with_ql_rewrites_trunk_953221.patch
>
>
> Implement indexing on Hive so that lookup and range queries are efficient.

-- 
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