----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53905/#review157504 -----------------------------------------------------------
Fix it, then Ship it! sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/TransactionManager.java (line 74) <https://reviews.apache.org/r/53905/#comment228111> Rename to failedTransactionsCount. - Hao Hao On Nov. 18, 2016, 10:50 p.m., Alexander Kolbasov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53905/ > ----------------------------------------------------------- > > (Updated Nov. 18, 2016, 10:50 p.m.) > > > Review request for sentry, Colm O hEigeartaigh, Colin Ma, Hao Hao, Sravya > Tirukkovalur, and Vadim Spector. > > > Repository: sentry > > > Description > ------- > > SENTRY-1518: Add metrics for SentryStore transactions > > > Diffs > ----- > > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/TransactionManager.java > 3d0a6ab0b514cb9fbead4b8e527f7cb84865ce28 > > Diff: https://reviews.apache.org/r/53905/diff/ > > > Testing > ------- > > > Thanks, > > Alexander Kolbasov > >
