qzsee commented on code in PR #38301:
URL: https://github.com/apache/doris/pull/38301#discussion_r1699770058


##########
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalCheckPolicy.java:
##########
@@ -230,4 +235,62 @@ public RelatedPolicy(Optional<Expression> rowPolicyFilter, 
Optional<List<NamedEx
             this.dataMaskProjects = dataMaskProjects;
         }
     }
+
+    private Expression getMaskTypeDef(NereidsParser parser, DataMaskPolicy 
dataMaskPolicy, Slot slot) {
+        if (dataMaskPolicy instanceof DorisDataMaskPolicy) {

Review Comment:
   Yes, it is better to put it in the data mask logic, but I did it because I 
don't want to change too much code. I'll put it in the data mask logic and 
change the code later.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to