----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39228/#review103614 -----------------------------------------------------------
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/SimpleDBProviderBackend.java (line 83) <https://reviews.apache.org/r/39228/#comment161684> How about? if(retry > 0) LOG.warn (msg + " Retrying..") else LOG.error - Sravya Tirukkovalur On Oct. 21, 2015, 11:56 a.m., Yibing Shi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39228/ > ----------------------------------------------------------- > > (Updated Oct. 21, 2015, 11:56 a.m.) > > > Review request for sentry and Sravya Tirukkovalur. > > > Repository: sentry > > > Description > ------- > > SENTRY-902: SimpleDBProviderBackend should retry the authrization process > properly > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/SimpleDBProviderBackend.java > 191e099 > > Diff: https://reviews.apache.org/r/39228/diff/ > > > Testing > ------- > > Test on a local cluster to make sure this retrying logic works. > > > Thanks, > > Yibing Shi > >
