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}";
}
}