You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com> on 2016/04/04 22:45:57 UTC
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/
-----------------------------------------------------------
(Updated April 4, 2016, 8:45 p.m.)
Review request for hive and Ashutosh Chauhan.
Repository: hive-git
Description
-------
Metastore Changes : API calls for retrieving primary keys and foreign keys information
Diffs (updated)
-----
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/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 ac293b9
metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
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/DummyRawStoreControlledCommit.java 94ca835
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
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
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com>.
> On April 4, 2016, 11:11 p.m., Ashutosh Chauhan wrote:
> > metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java, line 2601
> > <https://reviews.apache.org/r/45406/diff/2/?file=1324927#file1324927line2601>
> >
> > We need to implement these methods.
Will do in a followup jira
- Hari Sankar
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/#review126950
-----------------------------------------------------------
On April 12, 2016, 8:47 p.m., Hari Sankar Sivarama Subramaniyan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45406/
> -----------------------------------------------------------
>
> (Updated April 12, 2016, 8:47 p.m.)
>
>
> Review request for hive and Ashutosh Chauhan.
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> 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/019-HIVE-13076.mssql.sql PRE-CREATION
> 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/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java c9fadad
> metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java 64a26ac
> 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 ac293b9
> metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
> metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
> metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraint.java PRE-CREATION
> metastore/src/model/package.jdo 7385a13
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java 94ca835
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
>
> Diff: https://reviews.apache.org/r/45406/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Hari Sankar Sivarama Subramaniyan
>
>
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Ashutosh Chauhan <ha...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/#review126950
-----------------------------------------------------------
metastore/if/hive_metastore.thrift (line 53)
<https://reviews.apache.org/r/45406/#comment190033>
We don't use term schema that much in Hive. So, its better to call it db
metastore/if/hive_metastore.thrift (line 61)
<https://reviews.apache.org/r/45406/#comment190034>
Similar comment to use db instead of schema.
metastore/if/hive_metastore.thrift (line 302)
<https://reviews.apache.org/r/45406/#comment190036>
its better to mark new fields as optional param for better compat.
metastore/if/hive_metastore.thrift (line 1215)
<https://reviews.apache.org/r/45406/#comment190037>
We should follow request/response pattern for these api design. e.g, get_aggr_stats_for()
metastore/scripts/upgrade/mysql/034-HIVE-13076.mysql.sql (lines 4 - 6)
<https://reviews.apache.org/r/45406/#comment190078>
Instead of storing name, type and comment. We should just only store CS_ID with not null constraint here.
metastore/scripts/upgrade/mysql/034-HIVE-13076.mysql.sql (line 7)
<https://reviews.apache.org/r/45406/#comment190079>
We also need to store constraint name.
metastore/scripts/upgrade/mysql/034-HIVE-13076.mysql.sql (line 8)
<https://reviews.apache.org/r/45406/#comment190080>
Also, it will be good to add index for this table on CS_ID
metastore/scripts/upgrade/mysql/034-HIVE-13076.mysql.sql (line 13)
<https://reviews.apache.org/r/45406/#comment190082>
There should be an index on this table on constraint_name.
metastore/scripts/upgrade/mysql/034-HIVE-13076.mysql.sql (line 21)
<https://reviews.apache.org/r/45406/#comment190083>
Index on constraint name.
metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreDirectSql.java (lines 1819 - 1821)
<https://reviews.apache.org/r/45406/#comment190084>
These joins could be expensive.
metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java (line 2601)
<https://reviews.apache.org/r/45406/#comment190085>
We need to implement these methods.
metastore/src/model/package.jdo (line 151)
<https://reviews.apache.org/r/45406/#comment190086>
Its better to store PKeys in COLUMNS_V2 and thus we can avoid a join.
- Ashutosh Chauhan
On April 4, 2016, 8:45 p.m., Hari Sankar Sivarama Subramaniyan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45406/
> -----------------------------------------------------------
>
> (Updated April 4, 2016, 8:45 p.m.)
>
>
> Review request for hive and Ashutosh Chauhan.
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> 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/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 ac293b9
> metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
> metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
> 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/DummyRawStoreControlledCommit.java 94ca835
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
> 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
>
>
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com>.
> On April 11, 2016, 11:23 p.m., Ashutosh Chauhan wrote:
> > metastore/scripts/upgrade/derby/034-HIVE-13076.derby.sql, line 2
> > <https://reviews.apache.org/r/45406/diff/3/?file=1339577#file1339577line2>
> >
> > Any reason for index on constraint name, position?
> > I think on CD_ID it makes more sense.
This is required for disallowing duplicate constraint names. Also, index is created on parent table id.
> On April 11, 2016, 11:23 p.m., Ashutosh Chauhan wrote:
> > metastore/scripts/upgrade/derby/034-HIVE-13076.derby.sql, line 1
> > <https://reviews.apache.org/r/45406/diff/3/?file=1339577#file1339577line1>
> >
> > Do you need TBL_ID?
> > Since you need a join with TBLS anyway and CD_ID is unique, I don't think its useful to store TBL_ID here.
Without TBL_ID, a join with SDS is required on SD_ID to retrieve all the columns associated with a table.
- Hari Sankar
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/#review128275
-----------------------------------------------------------
On April 12, 2016, 8:47 p.m., Hari Sankar Sivarama Subramaniyan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45406/
> -----------------------------------------------------------
>
> (Updated April 12, 2016, 8:47 p.m.)
>
>
> Review request for hive and Ashutosh Chauhan.
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> 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/019-HIVE-13076.mssql.sql PRE-CREATION
> 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/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java c9fadad
> metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java 64a26ac
> 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 ac293b9
> metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
> metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
> metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraint.java PRE-CREATION
> metastore/src/model/package.jdo 7385a13
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java 94ca835
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
>
> Diff: https://reviews.apache.org/r/45406/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Hari Sankar Sivarama Subramaniyan
>
>
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Ashutosh Chauhan <ha...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/#review128275
-----------------------------------------------------------
metastore/if/hive_metastore.thrift (line 1209)
<https://reviews.apache.org/r/45406/#comment191702>
I am not sure if this is needed.
metastore/scripts/upgrade/derby/034-HIVE-13076.derby.sql (line 1)
<https://reviews.apache.org/r/45406/#comment191710>
Do you need TBL_ID?
Since you need a join with TBLS anyway and CD_ID is unique, I don't think its useful to store TBL_ID here.
metastore/scripts/upgrade/derby/034-HIVE-13076.derby.sql (line 2)
<https://reviews.apache.org/r/45406/#comment191713>
Any reason for index on constraint name, position?
I think on CD_ID it makes more sense.
metastore/scripts/upgrade/mssql/hive-schema-2.1.0.mssql.sql (line 996)
<https://reviews.apache.org/r/45406/#comment191714>
Its better to just inline those statements.
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java (line 6141)
<https://reviews.apache.org/r/45406/#comment191715>
I don't think there is need for this.
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java (line 6170)
<https://reviews.apache.org/r/45406/#comment191716>
No need for this.
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java (line 6173)
<https://reviews.apache.org/r/45406/#comment191717>
Neither for this.
- Ashutosh Chauhan
On April 11, 2016, 5:45 p.m., Hari Sankar Sivarama Subramaniyan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45406/
> -----------------------------------------------------------
>
> (Updated April 11, 2016, 5:45 p.m.)
>
>
> Review request for hive and Ashutosh Chauhan.
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> 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/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/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/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/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 ac293b9
> metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
> metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
> metastore/src/model/package.jdo 7385a13
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java 94ca835
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
> service/src/gen/thrift/gen-py/hive_service/ThriftHive-remote 9a2322f
>
> Diff: https://reviews.apache.org/r/45406/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Hari Sankar Sivarama Subramaniyan
>
>
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Ashutosh Chauhan <ha...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/#review129030
-----------------------------------------------------------
metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java (line 3267)
<https://reviews.apache.org/r/45406/#comment192495>
Instead of making this check, we should just try to persist the row in constraint table, it will fail in case there is a duplicate name, since its a part of primary key, in which case we pick another name.
metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraintPK.java (line 7)
<https://reviews.apache.org/r/45406/#comment192496>
I don't see this class getting used anywhere. In jdo mapping its seems you are using inner class PK which is defined in MConstraint. Do we need both?
- Ashutosh Chauhan
On April 14, 2016, 10:44 p.m., Hari Sankar Sivarama Subramaniyan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45406/
> -----------------------------------------------------------
>
> (Updated April 14, 2016, 10:44 p.m.)
>
>
> Review request for hive and Ashutosh Chauhan.
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> 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/019-HIVE-13076.mssql.sql PRE-CREATION
> 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-javabean/org/apache/hadoop/hive/metastore/api/ForeignKeysRequest.java PRE-CREATION
> metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ForeignKeysResponse.java PRE-CREATION
> metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PrimaryKeysRequest.java PRE-CREATION
> metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PrimaryKeysResponse.java PRE-CREATION
> metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/SQLForeignKey.java PRE-CREATION
> metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/SQLPrimaryKey.java PRE-CREATION
> metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java c9fadad
> metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java 64a26ac
> 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 ac293b9
> metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
> metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
> metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraint.java PRE-CREATION
> metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraintPK.java PRE-CREATION
> metastore/src/model/package.jdo 7385a13
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java 94ca835
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
>
> Diff: https://reviews.apache.org/r/45406/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Hari Sankar Sivarama Subramaniyan
>
>
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/
-----------------------------------------------------------
(Updated April 14, 2016, 10:44 p.m.)
Review request for hive and Ashutosh Chauhan.
Changes
-------
1. Added enable/validate/rely bits to the metastore.
2. Validate fk name, pk names check for duplicates.
3. Generate constraint names if the user does not provide one.
Repository: hive-git
Description
-------
Metastore Changes : API calls for retrieving primary keys and foreign keys information
Diffs (updated)
-----
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/019-HIVE-13076.mssql.sql PRE-CREATION
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-javabean/org/apache/hadoop/hive/metastore/api/ForeignKeysRequest.java PRE-CREATION
metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ForeignKeysResponse.java PRE-CREATION
metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PrimaryKeysRequest.java PRE-CREATION
metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/PrimaryKeysResponse.java PRE-CREATION
metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/SQLForeignKey.java PRE-CREATION
metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/SQLPrimaryKey.java PRE-CREATION
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java c9fadad
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java 64a26ac
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 ac293b9
metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraint.java PRE-CREATION
metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraintPK.java PRE-CREATION
metastore/src/model/package.jdo 7385a13
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java 94ca835
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
Diff: https://reviews.apache.org/r/45406/diff/
Testing
-------
Thanks,
Hari Sankar Sivarama Subramaniyan
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/
-----------------------------------------------------------
(Updated April 13, 2016, 7:59 p.m.)
Review request for hive and Ashutosh Chauhan.
Repository: hive-git
Description
-------
Metastore Changes : API calls for retrieving primary keys and foreign keys information
Diffs (updated)
-----
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/019-HIVE-13076.mssql.sql PRE-CREATION
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/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java c9fadad
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java 64a26ac
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 ac293b9
metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraint.java PRE-CREATION
metastore/src/model/package.jdo 7385a13
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java 94ca835
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
Diff: https://reviews.apache.org/r/45406/diff/
Testing
-------
Thanks,
Hari Sankar Sivarama Subramaniyan
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com>.
> On April 13, 2016, 3:12 p.m., Ashutosh Chauhan wrote:
> > In hive-schema-2.1-***.sql instead of adding run <script-name> you may just inline new create table statements which is consistent with other create table statements there.
Ok makes sense. Will do it in the new upload.
> On April 13, 2016, 3:12 p.m., Ashutosh Chauhan wrote:
> > metastore/if/hive_metastore.thrift, lines 62-63
> > <https://reviews.apache.org/r/45406/diff/4/?file=1341804#file1341804line62>
> >
> > fk_constraint_name, pk_constraint_name ?
The naming convention is based on https://docs.oracle.com/javase/7/docs/api/java/sql/DatabaseMetaData.html#getPrimaryKeys(java.lang.String,%20java.lang.String,%20java.lang.String).
> On April 13, 2016, 3:12 p.m., Ashutosh Chauhan wrote:
> > metastore/if/hive_metastore.thrift, line 49
> > <https://reviews.apache.org/r/45406/diff/4/?file=1341804#file1341804line49>
> >
> > should this be called constraint name ?
https://docs.oracle.com/javase/7/docs/api/java/sql/DatabaseMetaData.html#getPrimaryKeys(java.lang.String,%20java.lang.String,%20java.lang.String) is the naming standard followed
> On April 13, 2016, 3:12 p.m., Ashutosh Chauhan wrote:
> > metastore/scripts/upgrade/mssql/019-HIVE-13076.mssql.sql, line 1
> > <https://reviews.apache.org/r/45406/diff/4/?file=1341808#file1341808line1>
> >
> > IF NOT Exists missing
Create table without an IF NOT EXISTS is in convention with the create statements in the *mssql.sql files. I added "IF EXISTS" on oracle/postgres scripts based on the comments. But in hindsight, it is not necessary/rather may be a bad approach to add the IF NOT Exists clause since we should throw error if the table already exists and not silently ignore the create statement which can cause issue if the already existing table definition is not the same as the one expected by hive.
- Hari Sankar
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/#review128681
-----------------------------------------------------------
On April 12, 2016, 8:47 p.m., Hari Sankar Sivarama Subramaniyan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45406/
> -----------------------------------------------------------
>
> (Updated April 12, 2016, 8:47 p.m.)
>
>
> Review request for hive and Ashutosh Chauhan.
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> 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/019-HIVE-13076.mssql.sql PRE-CREATION
> 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/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java c9fadad
> metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java 64a26ac
> 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 ac293b9
> metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
> metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
> metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraint.java PRE-CREATION
> metastore/src/model/package.jdo 7385a13
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java 94ca835
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
>
> Diff: https://reviews.apache.org/r/45406/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Hari Sankar Sivarama Subramaniyan
>
>
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Ashutosh Chauhan <ha...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/#review128681
-----------------------------------------------------------
In hive-schema-2.1-***.sql instead of adding run <script-name> you may just inline new create table statements which is consistent with other create table statements there.
metastore/if/hive_metastore.thrift (line 49)
<https://reviews.apache.org/r/45406/#comment192159>
should this be called constraint name ?
metastore/if/hive_metastore.thrift (lines 60 - 61)
<https://reviews.apache.org/r/45406/#comment192162>
Where is this info store in backend?
metastore/if/hive_metastore.thrift (lines 62 - 63)
<https://reviews.apache.org/r/45406/#comment192161>
fk_constraint_name, pk_constraint_name ?
metastore/scripts/upgrade/mssql/019-HIVE-13076.mssql.sql (line 1)
<https://reviews.apache.org/r/45406/#comment192155>
IF NOT Exists missing
metastore/scripts/upgrade/mssql/019-HIVE-13076.mssql.sql (line 7)
<https://reviews.apache.org/r/45406/#comment192148>
In MTable desc, it is int so its better to store it as int
metastore/scripts/upgrade/mysql/hive-schema-2.1.0.mysql.sql (line 822)
<https://reviews.apache.org/r/45406/#comment192163>
inline statements here
metastore/scripts/upgrade/oracle/034-HIVE-13076.oracle.sql (line 1)
<https://reviews.apache.org/r/45406/#comment192156>
if not exists
metastore/scripts/upgrade/postgres/033-HIVE-13076.postgres.sql (line 1)
<https://reviews.apache.org/r/45406/#comment192158>
if not exists ?
- Ashutosh Chauhan
On April 12, 2016, 8:47 p.m., Hari Sankar Sivarama Subramaniyan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45406/
> -----------------------------------------------------------
>
> (Updated April 12, 2016, 8:47 p.m.)
>
>
> Review request for hive and Ashutosh Chauhan.
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> 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/019-HIVE-13076.mssql.sql PRE-CREATION
> 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/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java c9fadad
> metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java 64a26ac
> 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 ac293b9
> metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
> metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
> metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraint.java PRE-CREATION
> metastore/src/model/package.jdo 7385a13
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java 94ca835
> metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
>
> Diff: https://reviews.apache.org/r/45406/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Hari Sankar Sivarama Subramaniyan
>
>
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/
-----------------------------------------------------------
(Updated April 12, 2016, 8:47 p.m.)
Review request for hive and Ashutosh Chauhan.
Repository: hive-git
Description
-------
Metastore Changes : API calls for retrieving primary keys and foreign keys information
Diffs (updated)
-----
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/019-HIVE-13076.mssql.sql PRE-CREATION
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/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java c9fadad
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java 64a26ac
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 ac293b9
metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
metastore/src/model/org/apache/hadoop/hive/metastore/model/MConstraint.java PRE-CREATION
metastore/src/model/package.jdo 7385a13
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java 94ca835
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
Diff: https://reviews.apache.org/r/45406/diff/
Testing
-------
Thanks,
Hari Sankar Sivarama Subramaniyan
Re: Review Request 45406: HIVE-13349: Metastore Changes : API calls
for retrieving primary keys and foreign keys information
Posted by Hari Sankar Sivarama Subramaniyan <hs...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45406/
-----------------------------------------------------------
(Updated April 11, 2016, 5:45 p.m.)
Review request for hive and Ashutosh Chauhan.
Repository: hive-git
Description
-------
Metastore Changes : API calls for retrieving primary keys and foreign keys information
Diffs (updated)
-----
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/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/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/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/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 ac293b9
metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java e49f757
metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java a73dbeb
metastore/src/model/package.jdo 7385a13
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java 94ca835
metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java b108f95
service/src/gen/thrift/gen-py/hive_service/ThriftHive-remote 9a2322f
Diff: https://reviews.apache.org/r/45406/diff/
Testing
-------
Thanks,
Hari Sankar Sivarama Subramaniyan