-----------------------------------------------------------
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.
Changes
-------
Addressed binary compatibility changes.
Defined a new structure TAuthzUpdateRequest so that we avoid creating a new
method for get_authz_updates() in case a new parameter is added.
Defined a constant value unused path image numbers.
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 (updated)
-----
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/
Changes: https://reviews.apache.org/r/59895/diff/2-3/
Testing
-------
Thanks,
Sergio Pena
