-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59566/
-----------------------------------------------------------
(Updated June 1, 2017, 2:56 a.m.)
Review request for sentry, Alexander Kolbasov, kalyan kumar kalvagadda, Sergio
Pena, and Vamsee Yarlagadda.
Bugs: SENTRY-1784
https://issues.apache.org/jira/browse/SENTRY-1784
Repository: sentry
Description
-------
HDFS starts requesting changeID = 0 instead of 1. DBUpdateForwarder returns
full update if the request changeID <= 0. After first full update, the request
changeID = 1, so only delta update is sent unless clean up removes changes that
are not sent to HDFS. This fixes both issues in this Jira.
Diffs (updated)
-----
sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ServiceConstants.java
0741ebc
sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/UpdateableAuthzPaths.java
ad7f8c9
sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java
90ba721
sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryUpdater.java
34caa0e
sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/UpdateableAuthzPermissions.java
431c7fe
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/DBUpdateForwarder.java
b8542b3
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
9beb07b
Diff: https://reviews.apache.org/r/59566/diff/3/
Changes: https://reviews.apache.org/r/59566/diff/2-3/
Testing
-------
TestHDFSIntegrationEnd2End
Thanks,
Na Li
