Quanlong Huang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16930 )

Change subject: IMPALA-10387: Add missing overloads of mask functions used in 
Ranger default masking policies
......................................................................


Patch Set 4: Code-Review+2

> Patch Set 3: Code-Review+2
>
> Raising to +2
> I thought a bit about the use cases, and it is probably common to mask most 
> columns first and than fine tune it if turns out that some info is actually 
> needed. This is made easier by having these otherwise not too useful 
> overloads.
>
> It would be nice to create + mention a ticket about 0001-01-01 issue

Sure. Created HIVE-24641 and mentioned it in comments.

Carrying on Csaba's +2


--
To view, visit http://gerrit.cloudera.org:8080/16930
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icf3e70fd7aa9f3b6d6b508b776696e61ec1fcc2e
Gerrit-Change-Number: 16930
Gerrit-PatchSet: 4
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Fang-Yu Rao <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Comment-Date: Fri, 15 Jan 2021 07:24:05 +0000
Gerrit-HasComments: No

Reply via email to