-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36582/
-----------------------------------------------------------
Review request for sentry, Colin Ma and Lenni Kuff.
Bugs: SENTRY-778
https://issues.apache.org/jira/browse/SENTRY-778
Repository: sentry
Description
-------
Use Hadoop's getPassword instead of get for retriving password which looks up
the credential providers for the password and falls back to password specified
in the config file.
- Also removed default password for Sentry db.
Diffs
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/DelegateSentryStore.java
6061ef2a6a703a90b95ca69b273dcc6f0131e4e0
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
d7937d0976fa7c7736e6d1e3c68eb690aaac9060
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java
69086d202e874537cec068a594081144fd18f9e0
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
0d775f1636b0320beffc231a768db61ed4140ff6
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/SentryStoreIntegrationBase.java
79510228e5401cff82b645061f295c3c3c041324
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
35319dbf7a4c6c0c84f47a89a53ea8b613245ef0
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryVersion.java
0add58b371ad139f9477d43aa8076f7f96def162
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/tools/TestSentrySchemaTool.java
9a2dff811dcd488698d9fb541ff970b764edc69d
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java
2eea07b42d58406fb75a95e31b8fc8f78f02d341
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java
0c9feabc2dd0006972eeee5df8e19dc9c43c1009
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java
53d71d6b1f89c26e2607b43063e2f8c28c322b1b
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
3a8a6efc352745972a83687196698d45c9e016c4
sentry-tests/sentry-tests-solr/src/test/java/org/apache/sentry/tests/e2e/solr/db/integration/AbstractSolrSentryTestWithDbProvider.java
9438ee5808cb82b83b185eb659109e2008515243
sentry-tests/sentry-tests-sqoop/src/test/java/org/apache/sentry/tests/e2e/sqoop/AbstractSqoopSentryTestBase.java
2c6f329bb0b9a458f308718747c166b2815b7881
Diff: https://reviews.apache.org/r/36582/diff/
Testing
-------
- Added a new test to make sure getPassword works.
- Updated existing tests
Thanks,
Sravya Tirukkovalur
