----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65224/#review195830 -----------------------------------------------------------
Thanks Lina, I see that javadoc is excluding thrift auto-generated classes now. Could you change the subject of this review to reflect the new change? - Sergio Pena On Jan. 19, 2018, 4:15 p.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65224/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2018, 4:15 p.m.) > > > Review request for sentry, Arjun Mishra, Brian Towles, kalyan kumar > kalvagadda, and Sergio Pena. > > > Repository: sentry > > > Description > ------- > > add path for generated sentry thrift files. And exclude generated thrift > files for javadoc generation > > > Diffs > ----- > > sentry-hdfs/sentry-hdfs-common/pom.xml 58719c3 > sentry-provider/sentry-provider-db/pom.xml db4808d > > > Diff: https://reviews.apache.org/r/65224/diff/2/ > > > Testing > ------- > > ran build with javadoc plugin. Without this change, the javadoc build failed > for sentry-provider-db and sentry-hdfs-common. With this change, the build > succeeded > > > Thanks, > > Na Li > >
