You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Tuong Trg <ir...@yahoo.com> on 2014/11/04 22:48:46 UTC
Review Request 27592: JIRA SENTRY-486: Add database password
obfuscation support for sentry-site.xml
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27592/
-----------------------------------------------------------
Review request for sentry.
Repository: sentry
Description
-------
Add database password obfuscation support for sentry-site.xml
Diffs
-----
conf/sentry-site.xml.service.template a49b221
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/conf/HiveAuthzConf.java 4d2a625
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java f6699d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java 69086d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java bc86963
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/tools/TestSentrySchemaTool.java 9a2dff8
Diff: https://reviews.apache.org/r/27592/diff/
Testing
-------
all test ran clean
add a simple test method in SentrySchemaTool.
Thanks,
Tuong Trg
Re: Review Request 27592: JIRA SENTRY-486: Add database password
obfuscation support for sentry-site.xml
Posted by Tuong <ir...@yahoo.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27592/
-----------------------------------------------------------
(Updated Nov. 6, 2014, 7:35 p.m.)
Review request for sentry, Prasad Mujumdar and Sravya Tirukkovalur.
Changes
-------
Updated the diff file to include the 2 new files.
Repository: sentry
Description
-------
JIRA SENTRY-486: Add database password obfuscation support for sentry-site.xml
Diffs (updated)
-----
conf/sentry-site.xml.service.template a49b221
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/conf/HiveAuthzConf.java 4d2a625
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/password/Base64Cryptor.java PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/password/ICryptor.java PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java f6699d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java 69086d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java bc86963
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/tools/TestSentrySchemaTool.java 9a2dff8
Diff: https://reviews.apache.org/r/27592/diff/
Testing
-------
all test ran clean
add a simple test method in SentrySchemaTool.
Thanks,
Tuong
Re: Review Request 27592: JIRA SENTRY-486: Add database password
obfuscation support for sentry-site.xml
Posted by Tuong <ir...@yahoo.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27592/
-----------------------------------------------------------
(Updated Nov. 5, 2014, 8:21 p.m.)
Review request for sentry, Prasad Mujumdar and Sravya Tirukkovalur.
Repository: sentry
Description (updated)
-------
JIRA SENTRY-486: Add database password obfuscation support for sentry-site.xml
Diffs
-----
conf/sentry-site.xml.service.template a49b221
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/conf/HiveAuthzConf.java 4d2a625
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java f6699d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java 69086d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java bc86963
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/tools/TestSentrySchemaTool.java 9a2dff8
Diff: https://reviews.apache.org/r/27592/diff/
Testing
-------
all test ran clean
add a simple test method in SentrySchemaTool.
Thanks,
Tuong
Re: Review Request 27592: JIRA SENTRY-486: Add database password
obfuscation support for sentry-site.xml
Posted by Tuong <ir...@yahoo.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27592/
-----------------------------------------------------------
(Updated Nov. 5, 2014, 8:21 p.m.)
Review request for sentry, Prasad Mujumdar and Sravya Tirukkovalur.
Repository: sentry
Description
-------
Add database password obfuscation support for sentry-site.xml
Diffs
-----
conf/sentry-site.xml.service.template a49b221
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/conf/HiveAuthzConf.java 4d2a625
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java f6699d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java 69086d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java bc86963
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/tools/TestSentrySchemaTool.java 9a2dff8
Diff: https://reviews.apache.org/r/27592/diff/
Testing
-------
all test ran clean
add a simple test method in SentrySchemaTool.
Thanks,
Tuong
Re: Review Request 27592: JIRA SENTRY-486: Add database password
obfuscation support for sentry-site.xml
Posted by Tuong Trg <ir...@yahoo.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27592/
-----------------------------------------------------------
(Updated Nov. 5, 2014, 7:35 p.m.)
Review request for sentry and Prasad Mujumdar.
Repository: sentry
Description
-------
Add database password obfuscation support for sentry-site.xml
Diffs
-----
conf/sentry-site.xml.service.template a49b221
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/conf/HiveAuthzConf.java 4d2a625
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java f6699d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java 69086d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java bc86963
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/tools/TestSentrySchemaTool.java 9a2dff8
Diff: https://reviews.apache.org/r/27592/diff/
Testing
-------
all test ran clean
add a simple test method in SentrySchemaTool.
Thanks,
Tuong Trg
Re: Review Request 27592: JIRA SENTRY-486: Add database password
obfuscation support for sentry-site.xml
Posted by Tuong Trg <ir...@yahoo.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27592/
-----------------------------------------------------------
(Updated Nov. 4, 2014, 9:49 p.m.)
Review request for sentry.
Repository: sentry
Description
-------
Add database password obfuscation support for sentry-site.xml
Diffs
-----
conf/sentry-site.xml.service.template a49b221
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/conf/HiveAuthzConf.java 4d2a625
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java f6699d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java 69086d2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java bc86963
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/tools/TestSentrySchemaTool.java 9a2dff8
Diff: https://reviews.apache.org/r/27592/diff/
Testing
-------
all test ran clean
add a simple test method in SentrySchemaTool.
Thanks,
Tuong Trg