----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71039/#review216451 -----------------------------------------------------------
Ship it! Ship It! - Arjun Mishra On July 9, 2019, 3 p.m., Arjun Mishra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71039/ > ----------------------------------------------------------- > > (Updated July 9, 2019, 3 p.m.) > > > Review request for sentry and kalyan kumar kalvagadda. > > > Bugs: SENTRY-2528 > https://issues.apache.org/jira/browse/SENTRY-2528 > > > Repository: sentry > > > Description > ------- > > When fetching a full snapshot we get the below error. This is a regression of > SENTRY-2301 > > 2019-07-07 23:07:39,677 ERROR > org.apache.sentry.service.thrift.SentryHMSClient: Snapshot created failed > java.util.IllegalFormatConversionException: f != java.lang.Long > at > java.util.Formatter$FormatSpecifier.failConversion(Formatter.java:4302) > at java.util.Formatter$FormatSpecifier.printFloat(Formatter.java:2806) > at java.util.Formatter$FormatSpecifier.print(Formatter.java:2753) > at java.util.Formatter.format(Formatter.java:2520) > at java.util.Formatter.format(Formatter.java:2455) > at java.lang.String.format(String.java:2940) > at > org.apache.sentry.service.thrift.FullUpdateInitializer.getFullHMSSnapshot(FullUpdateInitializer.java:552) > at > org.apache.sentry.service.thrift.SentryHMSClient.fetchFullUpdate(SentryHMSClient.java:244) > at > org.apache.sentry.service.thrift.SentryHMSClient.getFullSnapshot(SentryHMSClient.java:147) > at > org.apache.sentry.provider.db.service.persistent.HMSFollower.createFullSnapshot(HMSFollower.java:409) > at > org.apache.sentry.provider.db.service.persistent.HMSFollower.syncupWithHms(HMSFollower.java:237) > at > org.apache.sentry.provider.db.service.persistent.HMSFollower.run(HMSFollower.java:198) > at > java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) > at java.util.concurrent.FutureTask.runAndReset(FutureTask.java:308) > at > java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$301(ScheduledThreadPoolExecutor.java:180) > at > java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:294) > at > java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) > at > java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) > at java.lang.Thread.run(Thread.java:748) > > > Diffs > ----- > > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/service/thrift/FullUpdateInitializer.java > d4bca42faea51d87164568c23bd7849c56450f32 > > > Diff: https://reviews.apache.org/r/71039/diff/1/ > > > Testing > ------- > > > Thanks, > > Arjun Mishra > >