----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49965/#review143041 -----------------------------------------------------------
metastore/if/hive_metastore.thrift (line 548) <https://reviews.apache.org/r/49965/#comment208737> you should provide default value here. 5: optional i32 numPartitions = -1 - Ashutosh Chauhan On July 20, 2016, 10:16 p.m., Hari Sankar Sivarama Subramaniyan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49965/ > ----------------------------------------------------------- > > (Updated July 20, 2016, 10:16 p.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Repository: hive-git > > > Description > ------- > > Hive generates inefficient metastore queries for TPCDS tables with 1800+ > partitions leading to higher compile time > > > Diffs > ----- > > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestHiveMetaStore.java > d90085b > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/hbase/TestHBaseAggrStatsCacheIntegration.java > 51d96dd > metastore/if/hive_metastore.thrift 4d92b73 > metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java > 38c0eed > > metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java > 909d8eb > metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java > b6fe502 > metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreDirectSql.java > 9c900af > metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java > 5adfa02 > metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java bbd47b8 > metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java > c65c7a4 > > metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java > 1ea72a0 > > metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java > 3e6acc7 > > 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 > ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java ef0bb3d > ql/src/java/org/apache/hadoop/hive/ql/optimizer/ppr/PartitionPruner.java > 26e936e > ql/src/java/org/apache/hadoop/hive/ql/parse/PrunedPartitionList.java > da2e1e2 > ql/src/java/org/apache/hadoop/hive/ql/stats/StatsUtils.java d8acf94 > > Diff: https://reviews.apache.org/r/49965/diff/ > > > Testing > ------- > > > Thanks, > > Hari Sankar Sivarama Subramaniyan > >