Return-Path: X-Original-To: apmail-hive-dev-archive@www.apache.org Delivered-To: apmail-hive-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id E7DEE18816 for ; Tue, 29 Mar 2016 00:12:08 +0000 (UTC) Received: (qmail 10520 invoked by uid 500); 29 Mar 2016 00:12:08 -0000 Delivered-To: apmail-hive-dev-archive@hive.apache.org Received: (qmail 10442 invoked by uid 500); 29 Mar 2016 00:12:08 -0000 Mailing-List: contact dev-help@hive.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hive.apache.org Delivered-To: mailing list dev@hive.apache.org Received: (qmail 10426 invoked by uid 99); 29 Mar 2016 00:12:08 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 29 Mar 2016 00:12:08 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 88DC128E494; Tue, 29 Mar 2016 00:12:06 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============2623734241421694138==" MIME-Version: 1.0 Subject: Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls for retrieving primary keys and foreign keys information From: Hari Sankar Sivarama Subramaniyan To: Ashutosh Chauhan Cc: Hari Sankar Sivarama Subramaniyan , hive Date: Tue, 29 Mar 2016 00:12:06 -0000 Message-ID: <20160329001206.30713.29106@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: Hari Sankar Sivarama Subramaniyan X-ReviewGroup: hive X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/45406/ X-Sender: Hari Sankar Sivarama Subramaniyan References: <20160329001107.30713.31249@reviews.apache.org> In-Reply-To: <20160329001107.30713.31249@reviews.apache.org> X-ReviewBoard-Diff-For: metastore/scripts/upgrade/derby/034-HIVE-13076.derby.sql X-ReviewBoard-Diff-For: metastore/src/model/org/apache/hadoop/hive/metastore/model/MPKFKMapping.java X-ReviewBoard-Diff-For: metastore/scripts/upgrade/postgres/033-HIVE-13076.postgres.sql X-ReviewBoard-Diff-For: metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ForeignKeyRel.java X-ReviewBoard-Diff-For: metastore/src/model/org/apache/hadoop/hive/metastore/model/MForeignKeyRel.java X-ReviewBoard-Diff-For: metastore/scripts/upgrade/mysql/034-HIVE-13076.mysql.sql X-ReviewBoard-Diff-For: metastore/scripts/upgrade/oracle/034-HIVE-13076.oracle.sql Reply-To: Hari Sankar Sivarama Subramaniyan X-ReviewRequest-Repository: hive-git --===============2623734241421694138== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45406/ ----------------------------------------------------------- (Updated March 29, 2016, 12:11 a.m.) Review request for hive and Ashutosh Chauhan. Repository: hive-git Description (updated) ------- Metastore Changes : API calls for retrieving primary keys and foreign keys information Diffs ----- metastore/if/hive_metastore.thrift 6a55962 metastore/scripts/upgrade/derby/034-HIVE-13076.derby.sql PRE-CREATION metastore/scripts/upgrade/derby/hive-schema-2.1.0.derby.sql 42f4eb6 metastore/scripts/upgrade/derby/upgrade-2.0.0-to-2.1.0.derby.sql a0bac3c metastore/scripts/upgrade/mssql/hive-schema-2.1.0.mssql.sql cf5a662 metastore/scripts/upgrade/mssql/upgrade-2.0.0-to-2.1.0.mssql.sql f25daf2 metastore/scripts/upgrade/mysql/034-HIVE-13076.mysql.sql PRE-CREATION metastore/scripts/upgrade/mysql/hive-schema-2.1.0.mysql.sql 6fd3209 metastore/scripts/upgrade/mysql/upgrade-2.0.0-to-2.1.0.mysql.sql e790636 metastore/scripts/upgrade/oracle/034-HIVE-13076.oracle.sql PRE-CREATION metastore/scripts/upgrade/oracle/hive-schema-2.1.0.oracle.sql 774f6be metastore/scripts/upgrade/oracle/upgrade-2.0.0-to-2.1.0.oracle.sql 8368d08 metastore/scripts/upgrade/postgres/033-HIVE-13076.postgres.sql PRE-CREATION metastore/scripts/upgrade/postgres/hive-schema-2.1.0.postgres.sql 7463a37 metastore/scripts/upgrade/postgres/upgrade-2.0.0-to-2.1.0.postgres.sql 6172407 metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore.h 8a8f8b1 metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore.cpp 6e5de20 metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore_server.skeleton.cpp 3e7c6e7 metastore/src/gen/thrift/gen-cpp/hive_metastore_types.h 97c07a5 metastore/src/gen/thrift/gen-cpp/hive_metastore_types.cpp 2695ffa metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/AddDynamicPartitions.java bb6e584 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/AddPartitionsRequest.java 083d340 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/AddPartitionsResult.java 9004457 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/AggrStats.java bfd1206 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ClearFileMetadataRequest.java 657bb7b metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ColumnStatistics.java ba059fe metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/Database.java 2769845 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/DropPartitionsResult.java adef415 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/EnvironmentContext.java de588e2 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/FireEventRequest.java 6b08234 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ForeignKeyRel.java PRE-CREATION metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/Function.java 5f8ce0d metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/GetAllFunctionsResponse.java f88e279 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/GetFileMetadataByExprRequest.java 0236b4a metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/GetFileMetadataByExprResult.java 89eb819 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/GetFileMetadataRequest.java 2408ad1 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/GetFileMetadataResult.java 8946635 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/GetOpenTxnsInfoResponse.java 629c042 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/GetOpenTxnsResponse.java 9f57a4a metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/GetPrincipalsInRoleResponse.java 02c652a metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/GetRoleGrantsForPrincipalResponse.java c013e0e metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/HeartbeatTxnRangeResponse.java b00fb9c metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/HiveObjectRef.java d7dd8ec metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/Index.java 44d99c7 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/InsertEventRequestData.java 488d3a0 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/LockRequest.java f39f582 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/NotificationEventResponse.java fcbbd18 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/OpenTxnsResponse.java e11a2b3 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/Partition.java a247221 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PartitionListComposingSpec.java e8232b9 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PartitionSpecWithSharedSD.java 2c41912 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PartitionWithoutSD.java f0c5ce1 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PartitionsByExprResult.java 12ae66d metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PartitionsStatsRequest.java 8416369 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PartitionsStatsResult.java 2e903f1 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PrincipalPrivilegeSet.java e1792bd metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PrivilegeBag.java bba2d40 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PutFileMetadataRequest.java ab151b1 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/RequestPartsSpec.java 7f8a044 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/Schema.java 75d0ebb metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/SerDeInfo.java d5a8d99 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/SetPartitionsStatsRequest.java 6e334f6 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ShowCompactResponse.java afa832c metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ShowLocksResponse.java b9b7f3c metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/SkewedInfo.java c32f50c metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/StorageDescriptor.java 938f06b metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/Table.java 5d683fb metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/TableStatsRequest.java d0daee5 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/TableStatsResult.java 78d4250 metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ThriftHiveMetastore.java 13e30db metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/Type.java 14cd42e metastore/src/gen/thrift/gen-php/metastore/ThriftHiveMetastore.php 05a0749 metastore/src/gen/thrift/gen-php/metastore/Types.php 488a920 metastore/src/gen/thrift/gen-py/hive_metastore/ThriftHiveMetastore-remote 516b926 metastore/src/gen/thrift/gen-py/hive_metastore/ThriftHiveMetastore.py ac8d8a4 metastore/src/gen/thrift/gen-py/hive_metastore/ttypes.py 10eaf4a metastore/src/gen/thrift/gen-rb/hive_metastore_types.rb 1cf40ae metastore/src/gen/thrift/gen-rb/thrift_hive_metastore.rb e782bb5 metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java c9fadad metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java cdd12ab metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java 39cf927 metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreDirectSql.java 06e9f78 metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java 35adb39 metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757 metastore/src/model/org/apache/hadoop/hive/metastore/model/MForeignKeyRel.java PRE-CREATION metastore/src/model/org/apache/hadoop/hive/metastore/model/MPKFKMapping.java PRE-CREATION metastore/src/model/org/apache/hadoop/hive/metastore/model/MTable.java 2a78ce9 metastore/src/model/package.jdo 7385a13 metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaStorePartitionSpecs.java 922a4bf metastore/src/test/org/apache/hadoop/hive/metastore/TestObjectStore.java 2e1f5f4 metastore/src/test/org/apache/hadoop/hive/metastore/hbase/TestHBaseAggregateStatsCache.java 6cd3a46 metastore/src/test/org/apache/hadoop/hive/metastore/hbase/TestHBaseAggregateStatsCacheWithBitVector.java e0c4094 metastore/src/test/org/apache/hadoop/hive/metastore/hbase/TestHBaseAggregateStatsExtrapolation.java f4e55ed metastore/src/test/org/apache/hadoop/hive/metastore/hbase/TestHBaseAggregateStatsNDVUniformDist.java 62918be metastore/src/test/org/apache/hadoop/hive/metastore/hbase/TestHBaseStore.java e4723f6 metastore/src/test/org/apache/hadoop/hive/metastore/hbase/TestHBaseStoreBitVector.java b1dc542 metastore/src/test/org/apache/hadoop/hive/metastore/hbase/TestHBaseStoreCached.java cfe9cd0 Diff: https://reviews.apache.org/r/45406/diff/ Testing ------- Thanks, Hari Sankar Sivarama Subramaniyan --===============2623734241421694138==--