---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47196/
---
(Updated May 11, 2016, 3:04 a.m.)
Review request for hive and Ashutosh Chauhan.
Changes
---
Added -ve test cases.
Repository: hive-git
Description
---
Support Alter commands for Rely/NoRely novalidate for PK/FK constraints
Diffs (updated)
-
metastore/if/hive_metastore.thrift 2eac836
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
8d4e8f3
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
3a63551
metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java
68972fb
metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java fea4fc5
metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java 07d8119
metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java
e97d4a9
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java
7f4b77e
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java
25ccced
ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 0204fcd
ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 981b961
ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java 7fcbd6a
ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 0a892e8
ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 685ab6a
ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzerFactory.java
23a7f6e
ql/src/java/org/apache/hadoop/hive/ql/plan/AlterTableDesc.java 38d8d5a
ql/src/java/org/apache/hadoop/hive/ql/plan/HiveOperation.java c6044b9
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveOperationType.java
810da48
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java
3f138fb
ql/src/test/queries/clientnegative/alter_table_constraint_duplicate_pk.q
PRE-CREATION
ql/src/test/queries/clientnegative/alter_table_constraint_invalid_fk_col1.q
PRE-CREATION
ql/src/test/queries/clientnegative/alter_table_constraint_invalid_fk_col2.q
PRE-CREATION
ql/src/test/queries/clientnegative/alter_table_constraint_invalid_fk_tbl1.q
PRE-CREATION
ql/src/test/queries/clientnegative/alter_table_constraint_invalid_fk_tbl2.q
PRE-CREATION
ql/src/test/queries/clientnegative/alter_table_constraint_invalid_pk_col.q
PRE-CREATION
ql/src/test/queries/clientnegative/alter_table_constraint_invalid_pk_tbl.q
PRE-CREATION
ql/src/test/queries/clientpositive/create_with_constraints.q 7aabf74
ql/src/test/results/clientnegative/alter_table_constraint_duplicate_pk.q.out
PRE-CREATION
ql/src/test/results/clientnegative/alter_table_constraint_invalid_fk_col1.q.out
PRE-CREATION
ql/src/test/results/clientnegative/alter_table_constraint_invalid_fk_col2.q.out
PRE-CREATION
ql/src/test/results/clientnegative/alter_table_constraint_invalid_fk_tbl1.q.out
PRE-CREATION
ql/src/test/results/clientnegative/alter_table_constraint_invalid_fk_tbl2.q.out
PRE-CREATION
ql/src/test/results/clientnegative/alter_table_constraint_invalid_pk_col.q.out
PRE-CREATION
ql/src/test/results/clientnegative/alter_table_constraint_invalid_pk_tbl.q.out
PRE-CREATION
ql/src/test/results/clientpositive/create_with_constraints.q.out 34dac30
Diff: https://reviews.apache.org/r/47196/diff/
Testing
---
Local testing done.
Thanks,
Hari Sankar Sivarama Subramaniyan
