be consistent with the log messages and use DSL.role() instead of DSL.name() where possible

Author: ThoSap

generated/build.gradle.kts

@@ -47,6 +47,18 @@ jooq {
                 | pg_auth_members
                 | pg_authid
                 | pg_db_role_setting
+                
+                | pg_database
+                | pg_namespace
+                | pg_class
+                | pg_proc
+                | pg_foreign_data_wrapper
+                | pg_foreign_server
+                | pg_type
+                | pg_language
+                | pg_parameter_acl
+                | pg_tablespace
+                
                 | shobj_description
                 """.trimIndent()
                 excludes = """

operator/src/main/java/it/aboutbits/postgresql/core/BaseReconciler.java

@@ -62,7 +62,7 @@ public Optional<ClusterConnection> getReferencedClusterConnection(
         //noinspection ConstantConditions
         if (clusterConnection == null) {
             log.error(
-                    "The specified ClusterConnection does not exist [clusterConnection={}/{}]",
+                    "The specified ClusterConnection does not exist [resource={}/{}]",
                     connectionNamespace,
                     connectionName
             );
@@ -75,7 +75,7 @@ public Optional<ClusterConnection> getReferencedClusterConnection(
 
         if (!Objects.equals(currentPhase, expectedPhase)) {
             log.warn(
-                    "The specified ClusterConnection is not ready yet [clusterConnection={}/{}]",
+                    "The specified ClusterConnection is not ready yet [resource={}/{}]",
                     connectionNamespace,
                     connectionName
             );

operator/src/main/java/it/aboutbits/postgresql/crd/role/RoleReconciler.java

@@ -71,7 +71,7 @@ public UpdateControl<Role> reconcile(
 
         if (clusterConnectionOptional.isEmpty()) {
             status.setPhase(CRPhase.PENDING)
-                    .setMessage("The specified ClusterConnection does not exist or is not ready yet [clusterRef=%s/%s]".formatted(
+                    .setMessage("The specified ClusterConnection does not exist or is not ready yet [resource=%s/%s]".formatted(
                             getResourceNamespaceOrOwn(resource, clusterRef.getNamespace()),
                             clusterRef.getName()
                     ));
@@ -155,7 +155,7 @@ public DeleteControl cleanup(
         );
 
         if (clusterConnectionOptional.isEmpty()) {
-            status.setMessage("The specified ClusterConnection no longer exists or is not ready yet [clusterRef=%s/%s]".formatted(
+            status.setMessage("The specified ClusterConnection no longer exists or is not ready yet [resource=%s/%s]".formatted(
                     getResourceNamespaceOrOwn(resource, clusterRef.getNamespace()),
                     clusterRef.getName()
             ));
@@ -179,7 +179,7 @@ public DeleteControl cleanup(
                     status.getPhase()
             );
 
-            status.setMessage("Deletion failed: " + e.getMessage());
+            status.setMessage("Deletion failed: %s".formatted(e.getMessage()));
 
             return DeleteControl.noFinalizerRemoval()
                     .rescheduleAfter(60, TimeUnit.SECONDS);

operator/src/main/java/it/aboutbits/postgresql/crd/role/RoleService.java

@@ -20,7 +20,6 @@
 import static org.jooq.impl.DSL.field;
 import static org.jooq.impl.DSL.keyword;
 import static org.jooq.impl.DSL.multiset;
-import static org.jooq.impl.DSL.name;
 import static org.jooq.impl.DSL.query;
 import static org.jooq.impl.DSL.role;
 import static org.jooq.impl.DSL.select;
@@ -419,7 +418,7 @@ private static Query buildCommentOnRole(
     ) {
         return query(
                 "comment on role {0} is {1}",
-                name(roleName),
+                role(roleName),
                 val(comment)
         );
     }