You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Madhan Neethiraj <ma...@apache.org> on 2016/04/20 09:45:36 UTC
Review Request 46429: RANGER-895: mask UDF implementations refactoring
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46429/
-----------------------------------------------------------
Review request for ranger, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.
Bugs: RANGER-895
https://issues.apache.org/jira/browse/RANGER-895
Repository: ranger
Description
-------
Implementation of various mask UDFs refactored for performance
Diffs
-----
agents-common/src/main/resources/service-defs/ranger-servicedef-hive.json 1d97843
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java 3c922c9
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/EncryptTransformer.java f002f9a
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskFirstNTransformer.java e37b3ef
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskLastNTransformer.java 18bc558
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskShowFirstNTransformer.java 26a4c32
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskShowLastNTransformer.java cb30e50
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskTransformer.java 64de392
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerBaseUdf.java 0f5a7b2
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerTransformer.java fff4ad5
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerTransformerAdapter.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerTransformerFactory.java 83f646a
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMask.java 91caa8b
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskFirstN.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskHash.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskLastN.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskShowFirstN.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskShowLastN.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/ShuffleTransformer.java ee5023a
Diff: https://reviews.apache.org/r/46429/diff/
Testing
-------
Thanks,
Madhan Neethiraj
Re: Review Request 46429: RANGER-895: mask UDF implementations
refactoring
Posted by Abhay Kulkarni <ak...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46429/#review129837
-----------------------------------------------------------
Ship it!
Ship It!
- Abhay Kulkarni
On April 20, 2016, 7:45 a.m., Madhan Neethiraj wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46429/
> -----------------------------------------------------------
>
> (Updated April 20, 2016, 7:45 a.m.)
>
>
> Review request for ranger, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.
>
>
> Bugs: RANGER-895
> https://issues.apache.org/jira/browse/RANGER-895
>
>
> Repository: ranger
>
>
> Description
> -------
>
> Implementation of various mask UDFs refactored for performance
>
>
> Diffs
> -----
>
> agents-common/src/main/resources/service-defs/ranger-servicedef-hive.json 1d97843
> hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java 3c922c9
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/EncryptTransformer.java f002f9a
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskFirstNTransformer.java e37b3ef
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskLastNTransformer.java 18bc558
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskShowFirstNTransformer.java 26a4c32
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskShowLastNTransformer.java cb30e50
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskTransformer.java 64de392
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerBaseUdf.java 0f5a7b2
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerTransformer.java fff4ad5
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerTransformerAdapter.java PRE-CREATION
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerTransformerFactory.java 83f646a
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMask.java 91caa8b
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskFirstN.java PRE-CREATION
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskHash.java PRE-CREATION
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskLastN.java PRE-CREATION
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskShowFirstN.java PRE-CREATION
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskShowLastN.java PRE-CREATION
> ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/ShuffleTransformer.java ee5023a
>
> Diff: https://reviews.apache.org/r/46429/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Madhan Neethiraj
>
>
Re: Review Request 46429: RANGER-895: mask UDF implementations
refactoring
Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46429/
-----------------------------------------------------------
(Updated May 3, 2016, 4:12 p.m.)
Review request for ranger, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.
Changes
-------
Hive service-def updated to reference built-in mask functions in Hive (HIVE-13568). Removed ranger-hive-utils from Hive plugin packaging.
Bugs: RANGER-895
https://issues.apache.org/jira/browse/RANGER-895
Repository: ranger
Description
-------
Implementation of various mask UDFs refactored for performance
Diffs (updated)
-----
agents-common/src/main/resources/service-defs/ranger-servicedef-hive.json 4bb2607
src/main/assembly/hive-agent.xml d79dcbf
Diff: https://reviews.apache.org/r/46429/diff/
Testing
-------
Thanks,
Madhan Neethiraj
Re: Review Request 46429: RANGER-895: mask UDF implementations
refactoring
Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46429/
-----------------------------------------------------------
(Updated April 20, 2016, 11:35 p.m.)
Review request for ranger, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.
Changes
-------
some more refactoring to move the support classes inside RangerBaseUdf
Bugs: RANGER-895
https://issues.apache.org/jira/browse/RANGER-895
Repository: ranger
Description
-------
Implementation of various mask UDFs refactored for performance
Diffs (updated)
-----
agents-common/src/main/resources/service-defs/ranger-servicedef-hive.json 1d97843
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuditHandler.java 0745d1d
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java 3c922c9
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/EncryptTransformer.java f002f9a
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskFirstNTransformer.java e37b3ef
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskLastNTransformer.java 18bc558
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskShowFirstNTransformer.java 26a4c32
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskShowLastNTransformer.java cb30e50
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/MaskTransformer.java 64de392
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerBaseUdf.java 0f5a7b2
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerTransformer.java fff4ad5
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerTransformerFactory.java 83f646a
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMask.java 91caa8b
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskFirstN.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskHash.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskLastN.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskShowFirstN.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskShowLastN.java PRE-CREATION
ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/ShuffleTransformer.java ee5023a
Diff: https://reviews.apache.org/r/46429/diff/
Testing
-------
Thanks,
Madhan Neethiraj