hadoop-hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Sichi (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HIVE-1498) support IDXPROPERTIES on CREATE/ALTER INDEX
Date Mon, 20 Sep 2010 20:59:34 GMT

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

John Sichi commented on HIVE-1498:

Good start.  A few notes:

* I had to use patch -p1 to get this to apply.  Please make sure it applies with patch -p0

* In Hive.createIndex, you should be adding the tblProps to tt (not to temp).  The only purpose
of temp is to check whether the new table name already exists (in which case an exception
is thrown).  tt is the newly created table descriptor to be added to the metastore.

* For the next patch candidate, can you also add support for ALTER INDEX index_name ON table_name
SET IDXPROPERTIES (...)?  Or open a followup JIRA issue for this.

* Go ahead and add unit test coverage even though it's not actually possible to verify the
IDXPROPERTIES results until HIVE-1497 goes in.  You can modify ql/src/test/queries/clientpositive/index_creation.q,
or create a new index_props.q.  Also, you should be able to verify the TBLPROPERTIES results
already by doing DESCRIBE EXTENDED on the new table created by CREATE INDEX.

* Cosmetic:  For if statements, put the opening curly brace on the same line as the keyword.
 You can run ant checkstyle and look in the report for the files you modified to see formatting
which needs corrected.

> -------------------------------------------
>                 Key: HIVE-1498
>                 URL: https://issues.apache.org/jira/browse/HIVE-1498
>             Project: Hadoop Hive
>          Issue Type: Improvement
>          Components: Indexing
>    Affects Versions: 0.7.0
>            Reporter: John Sichi
>            Assignee: Russell Melick
>             Fix For: 0.7.0
>         Attachments: hive-1498.prelim.patch
> It's partially there in the grammar but not hooked in; should work pretty much the same

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

View raw message