----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59895/#review177691 -----------------------------------------------------------
Ship it! Ship It! - Alexander Kolbasov On June 12, 2017, 4:23 p.m., Sergio Pena wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59895/ > ----------------------------------------------------------- > > (Updated June 12, 2017, 4:23 p.m.) > > > Review request for sentry, Alexander Kolbasov, kalyan kumar kalvagadda, and > Vamsee Yarlagadda. > > > Bugs: SENTRY-1782 > https://issues.apache.org/jira/browse/SENTRY-1782 > > > Repository: sentry > > > Description > ------- > > This adds the HMS image ID to the hdfs/sentry thrift definition. The value > will be used as part of the hdfs/sentry updates requests in order to detect > if a full paths image should be returned or not. > > > Diffs > ----- > > > sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/SentryHDFSService.java > 29178d90b744b4b82b92ea24b1dc4f06dbebe36a > > sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TAuthzUpdateRequest.java > PRE-CREATION > > sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPathsUpdate.java > f3014be9fec01bb32fa171d27bed9ac23bac4717 > > sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/sentry_hdfs_serviceConstants.java > PRE-CREATION > > sentry-hdfs/sentry-hdfs-common/src/main/resources/sentry_hdfs_service.thrift > d01841b3bc4a8b38f44e08b45449e11352f6c725 > > sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceClientDefaultImpl.java > 1cdbb857f6d9c41684346da97431d011b2a19790 > > sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceProcessor.java > 395618aa7271b1b739a08f98d284f12c37d31a7c > > > Diff: https://reviews.apache.org/r/59895/diff/3/ > > > Testing > ------- > > > Thanks, > > Sergio Pena > >
