> On June 28, 2017, 8:36 p.m., Na Li wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryMetrics.java > > Lines 308 (patched) > > <https://reviews.apache.org/r/60490/diff/1/?file=1765924#file1765924line308> > > > > Can you call SentryServiceUtil.shutdownAndAwaitTermination to terminate > > the executor?
Sure, probably can do this. - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60490/#review179166 ----------------------------------------------------------- On June 28, 2017, 5:56 a.m., Alexander Kolbasov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60490/ > ----------------------------------------------------------- > > (Updated June 28, 2017, 5:56 a.m.) > > > Review request for sentry, Brian Towles, kalyan kumar kalvagadda, Na Li, > Sergio Pena, and Vamsee Yarlagadda. > > > Bugs: SENTRY-1820 > https://issues.apache.org/jira/browse/SENTRY-1820 > > > Repository: sentry > > > Description > ------- > > SENTRY-1820: Add JSON file reporter for Sentry metrics > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryMetrics.java > e3691a96ef1142fdae74ba873683a025265cc006 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java > d3c96fafc262ca7cfc3ba97ea24c715ab0b129ac > > > Diff: https://reviews.apache.org/r/60490/diff/1/ > > > Testing > ------- > > > Thanks, > > Alexander Kolbasov > >
