> On Sept. 26, 2018, 7:24 p.m., Alexander Kolbasov wrote: > > Are there any other cases where environment context isn't passed to the > > listener?
In other cases, it is being passed to listener > On Sept. 26, 2018, 7:24 p.m., Alexander Kolbasov wrote: > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java > > Line 746 (original), 746 (patched) > > <https://reviews.apache.org/r/68828/diff/1/?file=2091792#file2091792line746> > > > > Looks like formatting is off here I think formatting is correct here, no bugs according to checkstyle - Bharathkrishna ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68828/#review209036 ----------------------------------------------------------- On Sept. 24, 2018, 8:42 p.m., Bharathkrishna Guruvayoor Murali wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68828/ > ----------------------------------------------------------- > > (Updated Sept. 24, 2018, 8:42 p.m.) > > > Review request for hive and Alexander Kolbasov. > > > Repository: hive-git > > > Description > ------- > > It will be useful to have the environmentContext passed to > DbNotificationListener in this case, to know if the alter happened due to a > stat change. > > > Diffs > ----- > > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java > f52ff91a8f2e7710801dcadc4a83ce454992a66a > > > Diff: https://reviews.apache.org/r/68828/diff/1/ > > > Testing > ------- > > > Thanks, > > Bharathkrishna Guruvayoor Murali > >