----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60950/#review180981 -----------------------------------------------------------
Looks good to me. Just a few minor comments. metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java Lines 283 (patched) <https://reviews.apache.org/r/60950/#comment256367> These do not seem unnecessary. Maybe replace with: ``` private static final ThreadLocal<HMSHandler> threadLocalHMSHandler = new ThreadLocal<>(); ``` ? metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java Lines 382 (patched) <https://reviews.apache.org/r/60950/#comment256372> Maybe we can move this line before 375? - Chao Sun On July 19, 2017, 8:49 p.m., PRASHANT GOLASH wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60950/ > ----------------------------------------------------------- > > (Updated July 19, 2017, 8:49 p.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > Added the code to notify meta listeners during shutdown. Shutdown would > eventually call cleanupRawStore (In both cases HMSHandler#close and > TServerEventHandler#DeleteContext), so called the notification code in that > function. > > > Diffs > ----- > > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java > fd4527e653 > metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java > 58b9044930 > > > Diff: https://reviews.apache.org/r/60950/diff/4/ > > > Testing > ------- > > Added unit test cases for the affected codepath. > > > Thanks, > > PRASHANT GOLASH > >