----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/878/#review858 -----------------------------------------------------------
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java <https://reviews.apache.org/r/878/#comment1877> Can we make this method parameterized to reduce the number of casts required? E.g. private <T> Collection <T> getPartition... We might have to do something like <String>getPartition... when making the call though. - Paul On 2011-06-16 23:30:02, Sohan Jain wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/878/ > ----------------------------------------------------------- > > (Updated 2011-06-16 23:30:02) > > > Review request for hive and Paul Yang. > > > Summary > ------- > > If a table has a large number of partitions, get_partition_names_ps() make > take a long time to execute, because we get all of the partition names from > the database. This is not very memory efficient, and the operation can be > pushed down to the JDO layer without getting all of the names first. > > > This addresses bug HIVE-2213. > https://issues.apache.org/jira/browse/HIVE-2213 > > > Diffs > ----- > > trunk/common/src/java/org/apache/hadoop/hive/common/FileUtils.java 1135227 > > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java > 1135227 > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java > 1135227 > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java > 1135227 > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/Warehouse.java > 1135227 > > trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaStore.java > 1135227 > > Diff: https://reviews.apache.org/r/878/diff > > > Testing > ------- > > Passes previous test cases for get_partition_names_ps() in TestHiveMetaStore. > > > Thanks, > > Sohan > >
