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

jshao pushed a commit to branch branch-1.0
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/branch-1.0 by this push:
     new 4cae9e733b [#8383] fix : add a missing comma in 
insertPolicyMetaOnDuplicateKeyUpdate in PolicyMetaPostgreSQLProvider.java 
(#8412)
4cae9e733b is described below

commit 4cae9e733b9ea68674c468fe07d6cdea764ff1a6
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Thu Sep 4 11:12:39 2025 +0800

    [#8383] fix : add a missing comma in insertPolicyMetaOnDuplicateKeyUpdate 
in PolicyMetaPostgreSQLProvider.java (#8412)
    
    ### What changes were proposed in this pull request?
    
    Added a missing comma in the SQL statement of
    insertPolicyMetaOnDuplicateKeyUpdate in
    PolicyMetaPostgreSQLProvider.java.
    
    ### Why are the changes needed?
    
    Without this comma, the SQL would fail or behave incorrectly when
    updating policy metadata in PostgreSQL. This fixes a small bug that can
    cause runtime errors.
    
    Fix: #8383
    
    ### Does this PR introduce _any_ user-facing change?
    
    There are no changes to APIs or configuration visible to end users.
    Hence, it doesn't introduce _any_ user-facing change
    
    ### How was this patch tested?
    
    Applied Spotless formatting
    Ran ./gradlew build -x test.
    
    Co-authored-by: Ashwil-Colaco <[email protected]>
    Co-authored-by: Jerry Shao <[email protected]>
---
 .../mapper/provider/postgresql/PolicyMetaPostgreSQLProvider.java        | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/postgresql/PolicyMetaPostgreSQLProvider.java
 
b/core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/postgresql/PolicyMetaPostgreSQLProvider.java
index 2f4a63e863..5b0a7ae421 100644
--- 
a/core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/postgresql/PolicyMetaPostgreSQLProvider.java
+++ 
b/core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/postgresql/PolicyMetaPostgreSQLProvider.java
@@ -75,7 +75,7 @@ public class PolicyMetaPostgreSQLProvider extends 
PolicyMetaBaseSQLProvider {
         + " metalake_id = #{policyPO.metalakeId},"
         + " audit_info = #{policyPO.auditInfo},"
         + " current_version = #{policyPO.currentVersion},"
-        + " last_version = #{policyPO.lastVersion}"
+        + " last_version = #{policyPO.lastVersion},"
         + " deleted_at = #{policyPO.deletedAt}";
   }
 }

Reply via email to