You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by "jiraposter@reviews.apache.org (JIRA)" <ji...@apache.org> on 2011/08/25 20:19:31 UTC

[jira] [Commented] (HIVE-1503) additional validation for CREATE INDEX

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

jiraposter@reviews.apache.org commented on HIVE-1503:
-----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1661/
-----------------------------------------------------------

Review request for hive, John Sichi and Ning Zhang.


Summary
-------

Add some more validation for CREATE INDEX statements


This addresses bug HIVE-1503.
    https://issues.apache.org/jira/browse/HIVE-1503


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java c031f40 
  ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 3054f76 

Diff: https://reviews.apache.org/r/1661/diff


Testing
-------


Thanks,

Syed



> additional validation for CREATE INDEX
> --------------------------------------
>
>                 Key: HIVE-1503
>                 URL: https://issues.apache.org/jira/browse/HIVE-1503
>             Project: Hive
>          Issue Type: Improvement
>          Components: Indexing
>    Affects Versions: 0.7.0
>            Reporter: John Sichi
>         Attachments: HIVE-1503.1.patch
>
>
> Cases that should fail:
> * handler has usesIndexTable=false, and user supplies table name/format
> * handler has usesIndexTable=true, and user supplies table format that is incompatible with handler (does compact handler have any restrictions?)
> * user tries to create index on external table (?)
> * user tries to create index on index table
> * user tries to create index on column of unsupported datatype (e.g. array/map/struct probably does not make sense)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira