You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ha...@apache.org on 2016/05/11 20:31:42 UTC

[12/12] hive git commit: HIVE-13350: Support Alter commands for Rely/NoRely novalidate for PK/FK constraints (Hari Subramaniyan, reviewed by Ashutosh Chauhan)

HIVE-13350: Support Alter commands for Rely/NoRely novalidate for PK/FK constraints (Hari Subramaniyan, reviewed by Ashutosh Chauhan)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/b36f6a3a
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/b36f6a3a
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/b36f6a3a

Branch: refs/heads/master
Commit: b36f6a3a27adbcfa1aa1024f12dc3fdbb9334b51
Parents: bad8525
Author: Hari Subramaniyan <ha...@apache.org>
Authored: Wed May 11 13:31:10 2016 -0700
Committer: Hari Subramaniyan <ha...@apache.org>
Committed: Wed May 11 13:31:10 2016 -0700

----------------------------------------------------------------------
 metastore/if/hive_metastore.thrift              |   12 +
 .../gen/thrift/gen-cpp/ThriftHiveMetastore.cpp  | 3400 ++++++++-----
 .../gen/thrift/gen-cpp/ThriftHiveMetastore.h    |  266 ++
 .../ThriftHiveMetastore_server.skeleton.cpp     |   10 +
 .../gen/thrift/gen-cpp/hive_metastore_types.cpp | 2274 +++++----
 .../gen/thrift/gen-cpp/hive_metastore_types.h   |   84 +
 .../hive/metastore/api/AbortTxnsRequest.java    |   32 +-
 .../metastore/api/AddDynamicPartitions.java     |   32 +-
 .../metastore/api/AddForeignKeyRequest.java     |  443 ++
 .../metastore/api/AddPartitionsRequest.java     |   36 +-
 .../hive/metastore/api/AddPartitionsResult.java |   36 +-
 .../metastore/api/AddPrimaryKeyRequest.java     |  443 ++
 .../metastore/api/ClearFileMetadataRequest.java |   32 +-
 .../metastore/api/DropPartitionsResult.java     |   36 +-
 .../hive/metastore/api/FireEventRequest.java    |   32 +-
 .../hadoop/hive/metastore/api/Function.java     |   36 +-
 .../metastore/api/GetAllFunctionsResponse.java  |   36 +-
 .../api/GetFileMetadataByExprRequest.java       |   32 +-
 .../api/GetFileMetadataByExprResult.java        |   48 +-
 .../metastore/api/GetFileMetadataRequest.java   |   32 +-
 .../metastore/api/GetFileMetadataResult.java    |   44 +-
 .../metastore/api/GetOpenTxnsInfoResponse.java  |   36 +-
 .../hive/metastore/api/GetOpenTxnsResponse.java |   32 +-
 .../api/HeartbeatTxnRangeResponse.java          |   64 +-
 .../metastore/api/InsertEventRequestData.java   |   32 +-
 .../hadoop/hive/metastore/api/LockRequest.java  |   36 +-
 .../api/NotificationEventResponse.java          |   36 +-
 .../hive/metastore/api/OpenTxnsResponse.java    |   32 +-
 .../metastore/api/PartitionsByExprResult.java   |   36 +-
 .../metastore/api/PartitionsStatsRequest.java   |   64 +-
 .../metastore/api/PartitionsStatsResult.java    |   76 +-
 .../metastore/api/PutFileMetadataRequest.java   |   64 +-
 .../hive/metastore/api/RequestPartsSpec.java    |   68 +-
 .../hive/metastore/api/ShowCompactResponse.java |   36 +-
 .../hive/metastore/api/ShowLocksResponse.java   |   36 +-
 .../hive/metastore/api/TableStatsRequest.java   |   32 +-
 .../hive/metastore/api/TableStatsResult.java    |   36 +-
 .../hive/metastore/api/ThriftHiveMetastore.java | 4506 +++++++++++++-----
 .../gen-php/metastore/ThriftHiveMetastore.php   | 1716 ++++---
 .../src/gen/thrift/gen-php/metastore/Types.php  |  896 ++--
 .../hive_metastore/ThriftHiveMetastore-remote   |   14 +
 .../hive_metastore/ThriftHiveMetastore.py       | 1254 +++--
 .../gen/thrift/gen-py/hive_metastore/ttypes.py  |  602 ++-
 .../gen/thrift/gen-rb/hive_metastore_types.rb   |   34 +
 .../gen/thrift/gen-rb/thrift_hive_metastore.rb  |  126 +
 .../hadoop/hive/metastore/HiveMetaStore.java    |   53 +
 .../hive/metastore/HiveMetaStoreClient.java     |   14 +
 .../hadoop/hive/metastore/IMetaStoreClient.java |    5 +
 .../hadoop/hive/metastore/ObjectStore.java      |  100 +-
 .../apache/hadoop/hive/metastore/RawStore.java  |    4 +
 .../hadoop/hive/metastore/hbase/HBaseStore.java |   12 +
 .../DummyRawStoreControlledCommit.java          |   12 +
 .../DummyRawStoreForJdoConnection.java          |   12 +
 .../org/apache/hadoop/hive/ql/exec/DDLTask.java |   17 +
 .../hadoop/hive/ql/metadata/ForeignKeyInfo.java |    4 +-
 .../apache/hadoop/hive/ql/metadata/Hive.java    |   17 +
 .../hive/ql/parse/BaseSemanticAnalyzer.java     |   39 +-
 .../hive/ql/parse/DDLSemanticAnalyzer.java      |   25 +-
 .../apache/hadoop/hive/ql/parse/HiveParser.g    |   10 +
 .../hive/ql/parse/SemanticAnalyzerFactory.java  |    2 +
 .../hadoop/hive/ql/plan/AlterTableDesc.java     |   43 +-
 .../hadoop/hive/ql/plan/HiveOperation.java      |    2 +
 .../authorization/plugin/HiveOperationType.java |    1 +
 .../plugin/sqlstd/Operation2Privilege.java      |    2 +
 .../alter_table_constraint_duplicate_pk.q       |    2 +
 .../alter_table_constraint_invalid_fk_col1.q    |    3 +
 .../alter_table_constraint_invalid_fk_col2.q    |    3 +
 .../alter_table_constraint_invalid_fk_tbl1.q    |    3 +
 .../alter_table_constraint_invalid_fk_tbl2.q    |    3 +
 .../alter_table_constraint_invalid_pk_col.q     |    2 +
 .../alter_table_constraint_invalid_pk_tbl.q     |    3 +
 .../clientpositive/create_with_constraints.q    |   26 +-
 .../alter_table_constraint_duplicate_pk.q.out   |   11 +
 ...alter_table_constraint_invalid_fk_col1.q.out |   19 +
 ...alter_table_constraint_invalid_fk_col2.q.out |   19 +
 ...alter_table_constraint_invalid_fk_tbl1.q.out |   19 +
 ...alter_table_constraint_invalid_fk_tbl2.q.out |   19 +
 .../alter_table_constraint_invalid_pk_col.q.out |   11 +
 .../alter_table_constraint_invalid_pk_tbl.q.out |   19 +
 .../create_with_constraints.q.out               |  304 +-
 80 files changed, 12265 insertions(+), 5835 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/b36f6a3a/metastore/if/hive_metastore.thrift
----------------------------------------------------------------------
diff --git a/metastore/if/hive_metastore.thrift b/metastore/if/hive_metastore.thrift
index 2eac836..84e9b6d 100755
--- a/metastore/if/hive_metastore.thrift
+++ b/metastore/if/hive_metastore.thrift
@@ -493,6 +493,14 @@ struct DropConstraintRequest {
   3: required string constraintname
 }
 
+struct AddPrimaryKeyRequest {
+  1: required list<SQLPrimaryKey> primaryKeyCols
+}
+
+struct AddForeignKeyRequest {
+  1: required list<SQLForeignKey> foreignKeyCols
+}
+
 // Return type for get_partitions_by_expr
 struct PartitionsByExprResult {
   1: required list<Partition> partitions,
@@ -994,6 +1002,10 @@ service ThriftHiveMetastore extends fb303.FacebookService
               4:NoSuchObjectException o4)
   void drop_constraint(1:DropConstraintRequest req)
       throws(1:NoSuchObjectException o1, 2:MetaException o3)
+  void add_primary_key(1:AddPrimaryKeyRequest req)
+      throws(1:NoSuchObjectException o1, 2:MetaException o2)
+  void add_foreign_key(1:AddForeignKeyRequest req)
+      throws(1:NoSuchObjectException o1, 2:MetaException o2)  
 
   // drops the table and all the partitions associated with it if the table has partitions
   // delete data (including partitions) if deleteData is set to true