----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17162/#review32597 -----------------------------------------------------------
metastore/if/hive_metastore.thrift <https://reviews.apache.org/r/17162/#comment61503> nit. some ws probs. metastore/src/java/org/apache/hadoop/hive/metastore/StatObjectConverter.java <https://reviews.apache.org/r/17162/#comment61510> Thanks for hiding it. I appreciate that :-P ql/src/java/org/apache/hadoop/hive/ql/optimizer/StatsOptimizer.java <https://reviews.apache.org/r/17162/#comment61505> Are you planning on doing this in this patch? If not I'd rather see a follow up jira and the jira number here rather than "TODO" ql/src/java/org/apache/hadoop/hive/ql/optimizer/StatsOptimizer.java <https://reviews.apache.org/r/17162/#comment61506> Ditto. And lol. ql/src/java/org/apache/hadoop/hive/ql/stats/StatsUtils.java <https://reviews.apache.org/r/17162/#comment61509> Remove comment? Seems like a minor thing. ql/src/test/results/clientpositive/metadataonly1.q.out <https://reviews.apache.org/r/17162/#comment61511> why has this changed? - Gunther Hagleitner On Jan. 23, 2014, 12:03 a.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17162/ > ----------------------------------------------------------- > > (Updated Jan. 23, 2014, 12:03 a.m.) > > > Review request for hive and Gunther Hagleitner. > > > Repository: hive-git > > > Description > ------- > > see jira > > > Diffs > ----- > > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestHiveMetaStore.java > 1b01238 > metastore/if/hive_metastore.thrift e4e816d > metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java > 58f9957 > > metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java > ed05790 > metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java > 4288781 > metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreDirectSql.java > 16f43e9 > metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java > 794fadd > metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java f5ea2ef > > metastore/src/java/org/apache/hadoop/hive/metastore/StatObjectConverter.java > PRE-CREATION > > metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java > 003dc9c > > metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java > 6dd0852 > > metastore/src/test/org/apache/hadoop/hive/metastore/VerifyingObjectStore.java > c683fc9 > ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java d32deea > ql/src/java/org/apache/hadoop/hive/ql/optimizer/StatsOptimizer.java 608bef2 > ql/src/java/org/apache/hadoop/hive/ql/parse/ParseContext.java b815ea2 > ql/src/java/org/apache/hadoop/hive/ql/stats/StatsUtils.java 384b49e > ql/src/test/results/clientpositive/metadataonly1.q.out 3500fd2 > > Diff: https://reviews.apache.org/r/17162/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
