----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59205/#review174907 -----------------------------------------------------------
metastore/src/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java Line 381 (original), 526 (patched) <https://reviews.apache.org/r/59205/#comment248177> I think we only need a dirty flag, it will be reset by the next refresh, not end of individual call. metastore/src/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java Lines 528 (patched) <https://reviews.apache.org/r/59205/#comment248178> If using dirty flag, databaseCacheLock can be removed, right? - Daniel Dai On May 14, 2017, 5:19 a.m., Vaibhav Gumashta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59205/ > ----------------------------------------------------------- > > (Updated May 14, 2017, 5:19 a.m.) > > > Review request for hive, Daniel Dai and Thejas Nair. > > > Bugs: HIVE-16579 > https://issues.apache.org/jira/browse/HIVE-16579 > > > Repository: hive-git > > > Description > ------- > > https://issues.apache.org/jira/browse/HIVE-16579 > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java d6a80ae > > itests/hcatalog-unit/src/test/java/org/apache/hive/hcatalog/listener/DummyRawStoreFailEvent.java > 91a3a38 > itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java b897ffa > metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreDirectSql.java > b96c27e > metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java > 870896c > metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java > ed19f42 > metastore/src/java/org/apache/hadoop/hive/metastore/RawStore.java c1af690 > metastore/src/java/org/apache/hadoop/hive/metastore/cache/CacheUtils.java > 668499b > metastore/src/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java > 5a187d8 > metastore/src/java/org/apache/hadoop/hive/metastore/cache/SharedCache.java > 7beee42 > metastore/src/java/org/apache/hadoop/hive/metastore/hbase/HBaseStore.java > 0c7d8bb > > metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java > f613c30 > > metastore/src/test/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java > 1720e37 > > metastore/src/test/org/apache/hadoop/hive/metastore/cache/TestCachedStore.java > 0ab20d6 > > > Diff: https://reviews.apache.org/r/59205/diff/4/ > > > Testing > ------- > > > Thanks, > > Vaibhav Gumashta > >