> On April 6, 2017, 2:02 p.m., Na Li wrote: > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/FullUpdateInitializer.java > > Line 137 (original), 135 (patched) > > <https://reviews.apache.org/r/58093/diff/1/?file=1681748#file1681748line141> > > > > do you want to print out the value of "retries +1"? Does "retries + 1" > > gives you want you want? > > Alexander Kolbasov wrote: > It does - otherwise I'll see that it failed after 0 retries
I am working on subsequent fix to the same file - will fix it there. - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58093/#review171022 ----------------------------------------------------------- On March 31, 2017, 5:07 a.m., Alexander Kolbasov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58093/ > ----------------------------------------------------------- > > (Updated March 31, 2017, 5:07 a.m.) > > > Review request for sentry, Lei Xu, Hao Hao, kalyan kumar kalvagadda, and Na > Li. > > > Bugs: SENTRY-1676 > https://issues.apache.org/jira/browse/SENTRY-1676 > > > Repository: sentry > > > Description > ------- > > SENTRY-1676: FullUpdateInitializer#createInitialUpdate should not throw > RuntimeExceptio > > > Diffs > ----- > > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/FullUpdateInitializer.java > 146cea2b9467ce82b69bbf402933b1aa350bcd46 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > 6c14f5e46aad4223347d8d057188d31efbb68ed8 > > > Diff: https://reviews.apache.org/r/58093/diff/1/ > > > Testing > ------- > > > Thanks, > > Alexander Kolbasov > >
