-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60168/
-----------------------------------------------------------
(Updated June 19, 2017, 10:03 p.m.)
Review request for sentry, Brian Towles, Misha Dmitriev, kalyan kumar
kalvagadda, Na Li, Sergio Pena, Vamsee Yarlagadda, and Vadim Spector.
Changes
-------
Addressed code review changes from Misha
Bugs: SENTRY-1806
https://issues.apache.org/jira/browse/SENTRY-1806
Repository: sentry
Description
-------
SENTRY-1806: Improve memory handling for HDFS sync
Diffs (updated)
-----
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/exception/SentryHdfsServiceException.java
6b09dc265324e574448dc2de0d59ad45fe481fb8
sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/DeltaRetriever.java
0e585933ab021dd12116c9a29fbe306cbdeedcb9
sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java
6b31f7a4bbff66f6640813774ef79958990d7a57
sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PermissionsUpdate.java
14a4a0f6372bb4016a3a1134b348f334273dea7c
sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryUpdater.java
7304fd8bb30a793aae2e6afcddf25c44fe5d226b
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/DBUpdateForwarder.java
b8542b3ed7c1e4062fd89c6de12cc79b3d941062
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PathDeltaRetriever.java
5425114d38434dc8e6b1e4776be631b9b37fc6af
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PermDeltaRetriever.java
a29fc749828d673f951994fbbc555fc7a00a714c
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PermImageRetriever.java
5964f17e6316f8a72e701068a0b086e77763819f
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceClient.java
49d2360a87399bfa4dfc5b2f5fd6f517dca2708b
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceClientDefaultImpl.java
fc8bf4f6bce9b7c58a9152d3a340dfb687f23598
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceProcessor.java
28d6f5baad590ceb00bfc0c3d50675d91879489a
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
8b19c88e52400ce096cf8f38fe2ca0313f7e46e1
Diff: https://reviews.apache.org/r/60168/diff/2/
Changes: https://reviews.apache.org/r/60168/diff/1-2/
Testing
-------
Thanks,
Alexander Kolbasov
