This is an automated email from the ASF dual-hosted git repository.

jmclean pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new 4d36ab191e [#8340][Improvement]Remove the redundant 
content().validate() (#8365)
4d36ab191e is described below

commit 4d36ab191e76031b8998c34f6ec2d6a7075aaf11
Author: Jackeyzhe <[email protected]>
AuthorDate: Sun Aug 31 09:59:48 2025 +0800

    [#8340][Improvement]Remove the redundant content().validate() (#8365)
    
    ### What changes were proposed in this pull request?
    Remove the redundant content().validate() in
    core/src/main/java/org/apache/gravitino/meta/PolicyEntity.java
    
    
    ### Why are the changes needed?
    
    
    Fix: #8340
    
    ### Does this PR introduce _any_ user-facing change?
    no
    
    
    ### How was this patch tested?
    use exist ut
    
    Co-authored-by: jackeyzhe <[email protected]>
---
 core/src/main/java/org/apache/gravitino/meta/PolicyEntity.java | 1 -
 1 file changed, 1 deletion(-)

diff --git a/core/src/main/java/org/apache/gravitino/meta/PolicyEntity.java 
b/core/src/main/java/org/apache/gravitino/meta/PolicyEntity.java
index e93b8edd11..6a1070489f 100644
--- a/core/src/main/java/org/apache/gravitino/meta/PolicyEntity.java
+++ b/core/src/main/java/org/apache/gravitino/meta/PolicyEntity.java
@@ -132,7 +132,6 @@ public class PolicyEntity implements Entity, Auditable, 
HasIdentifier {
   public void validate() throws IllegalArgumentException {
     Entity.super.validate();
     validatePolicy();
-    content().validate();
   }
 
   @Override

Reply via email to