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

mchades 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 751672b704 [#8383] fix : add a missing comma in 
insertPolicyMetaOnDuplicateKeyUpdate in PolicyMetaPostgreSQLProvider.java 
(#8406)
751672b704 is described below

commit 751672b70424d430832c437a83f95188ba71d7ae
Author: Ashwil-Colaco <[email protected]>
AuthorDate: Wed Sep 3 13:56:36 2025 +0530

    [#8383] fix : add a missing comma in insertPolicyMetaOnDuplicateKeyUpdate 
in PolicyMetaPostgreSQLProvider.java (#8406)
    
    ### 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.
---
 .../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