----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58913/#review173587 -----------------------------------------------------------
Ship it! Ship It! - Na Li On May 2, 2017, 3:39 a.m., Vamsee Yarlagadda wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58913/ > ----------------------------------------------------------- > > (Updated May 2, 2017, 3:39 a.m.) > > > Review request for sentry, Alexander Kolbasov, Hao Hao, and kalyan kumar > kalvagadda. > > > Repository: sentry > > > Description > ------- > > With the recent refactoring change SENTRY-1639, we introduced a new class > SentryClientTransportConfigInterface. While trying to spin up a real cluster > with this change, NN could fail to start with NoClassDefFoundError as we only > put the jars under SENTRY_HOME/lib/plugins on the HDFS classpath and this new > class is part of sentry-core-common which is not bundled inside plugins/. > > > Diffs > ----- > > sentry-hdfs/sentry-hdfs-dist/pom.xml dd2d847 > > > Diff: https://reviews.apache.org/r/58913/diff/1/ > > > Testing > ------- > > Verified to see the sentry-core-common being bundled. > > > Thanks, > > Vamsee Yarlagadda > >
