----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58854/#review173398 -----------------------------------------------------------
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java Lines 189 (patched) <https://reviews.apache.org/r/58854/#comment246396> Do we support this style of format string? In other places we use {}-style formatting. - Alexander Kolbasov On April 28, 2017, 8:22 p.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58854/ > ----------------------------------------------------------- > > (Updated April 28, 2017, 8:22 p.m.) > > > Review request for sentry and Alexander Kolbasov. > > > Repository: sentry > > > Description > ------- > > add debug log message to help find root cause of SENTRY-1400 > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java > dfd79ae > > > Diff: https://reviews.apache.org/r/58854/diff/1/ > > > Testing > ------- > > > Thanks, > > Na Li > >
