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 2: Code-Review+1 (2 comments) lgtm I don't agree with some of the behaviors (e.g. date->0001-01-01 in mask/last_4/first_4), but if Hive works this way, then we should follow it. http://gerrit.cloudera.org:8080/#/c/16930/2/tests/authorization/test_ranger.py File tests/authorization/test_ranger.py: http://gerrit.cloudera.org:8080/#/c/16930/2/tests/authorization/test_ranger.py@1024 PS2, Line 1024: massking typo http://gerrit.cloudera.org:8080/#/c/16930/2/tests/authorization/test_ranger.py@1030 PS2, Line 1030: massking typo -- 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: 2 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: Mon, 11 Jan 2021 23:50:36 +0000 Gerrit-HasComments: Yes
