-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56954/
-----------------------------------------------------------
(Updated March 8, 2017, 6:41 p.m.)
Review request for sentry, Alexander Kolbasov, Hao Hao, Mat Crocker, Vamsee
Yarlagadda, and Vadim Spector.
Changes
-------
Addressed sasha review comments
Bugs: SENTRY-1639
https://issues.apache.org/jira/browse/SENTRY-1639
Repository: sentry
Description
-------
SENTRY-1639 Refactored the usage of configuration constants related to
transport These are subset of changes done for SENTRY-1592.
Diffs (updated)
-----
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/exception/MissingConfigurationException.java
PRE-CREATION
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryClientTransportConfigInterface.java
PRE-CREATION
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryClientTransportConstants.java
PRE-CREATION
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryHDFSClientTransportConfig.java
PRE-CREATION
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryPolicyClientTransportConfig.java
PRE-CREATION
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/SentryConstants.java
4ed1361
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceClientDefaultImpl.java
03bf39e
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceClientDefaultImpl.java
ee6cdf7
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClientDefaultImpl.java
2dc8af8
Diff: https://reviews.apache.org/r/56954/diff/4/
Changes: https://reviews.apache.org/r/56954/diff/3-4/
Testing
-------
Thanks,
kalyan kumar kalvagadda
