Csaba Ringhofer 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 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


--
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: 3
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: Thu, 14 Jan 2021 10:38:38 +0000
Gerrit-HasComments: No

Reply via email to