Polish

Author: nakamura-to

deployment/src/main/java/org/seasar/doma/quarkus/deployment/DomaClassScanner.java

@@ -19,7 +19,7 @@
 
 public class DomaClassScanner {
 
-  private static final Logger LOGGER = Logger.getLogger(DomaClassScanner.class);
+  private static final Logger logger = Logger.getLogger(DomaClassScanner.class);
 
   private static final DotName ENTITY = DotName.createSimple(Entity.class.getName());
   private static final DotName EMBEDDABLE = DotName.createSimple(Embeddable.class.getName());
@@ -48,7 +48,7 @@ List<String> scan() {
         .map(AnnotationTarget::asClass)
         .map(ClassInfo::name)
         .map(DotName::toString)
-        .peek(it -> LOGGER.debugf("class found: %s", it))
+        .peek(it -> logger.debugf("class found: %s", it))
         .collect(toList());
   }
 }

deployment/src/main/java/org/seasar/doma/quarkus/deployment/DomaResourceScanner.java

@@ -17,7 +17,8 @@
 
 public class DomaResourceScanner {
 
-  private static final Logger LOGGER = Logger.getLogger(DomaResourceScanner.class);
+  private static final Logger logger = Logger.getLogger(DomaResourceScanner.class);
+
   private static final String META_INF = "META-INF";
 
   List<String> scan() {
@@ -39,7 +40,7 @@ private void collect(Path resource, Set<String> files) {
             public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) {
               var fileName = resource.relativize(file).toString();
               if (fileName.endsWith(".sql") || fileName.endsWith(".script")) {
-                LOGGER.debugf("resource found: %s", fileName);
+                logger.debugf("resource found: %s", fileName);
                 files.add(fileName);
               }
               return FileVisitResult.CONTINUE;

deployment/src/main/java/org/seasar/doma/quarkus/deployment/DomaSettingsFactory.java

@@ -18,7 +18,7 @@
 
 public class DomaSettingsFactory {
 
-  private static final Logger LOGGER = Logger.getLogger(DomaSettingsFactory.class);
+  private static final Logger logger = Logger.getLogger(DomaSettingsFactory.class);
 
   private final DomaBuildTimeConfig buildTimeConfig;
   private final List<JdbcDataSourceBuildItem> dataSources;
@@ -46,7 +46,7 @@ DomaSettings create() {
     if (dataSources.isEmpty()) {
       throw new IllegalStateException("The quarkus.datasource is empty. Specify it.");
     }
-    LOGGER.debugf("settings: %s", settings);
+    logger.debugf("settings: %s", settings);
     return settings;
   }
 

runtime/src/main/java/org/seasar/doma/quarkus/runtime/JtaRequiresNewController.java

@@ -8,6 +8,7 @@
 @Singleton
 @DefaultBean
 public class JtaRequiresNewController implements RequiresNewController {
+
   @Override
   @Transactional(Transactional.TxType.REQUIRES_NEW)
   public <R> R requiresNew(Callback<R> callback) throws Throwable {

runtime/src/main/java/org/seasar/doma/quarkus/runtime/LogPreferences.java

@@ -6,10 +6,6 @@ public class LogPreferences {
   private final boolean dao;
   private final boolean closingFailure;
 
-  public LogPreferences() {
-    this(false, false, false);
-  }
-
   public LogPreferences(boolean sql, boolean dao, boolean closingFailure) {
     this.sql = sql;
     this.dao = dao;

runtime/src/main/java/org/seasar/doma/quarkus/runtime/ScriptExecutor.java

@@ -18,7 +18,7 @@
 @DefaultBean
 public class ScriptExecutor {
 
-  private static final Logger LOGGER = Logger.getLogger(ScriptExecutor.class.getName());
+  private static final Logger logger = Logger.getLogger(ScriptExecutor.class.getName());
 
   private final Map<String, String> namedSqlLoadScripts;
 
@@ -42,7 +42,7 @@ void onStartup(@Observes StartupEvent event) throws Exception {
   }
 
   private void execute(DataSource dataSource, String path) throws Exception {
-    LOGGER.infof("Execute %s", path);
+    logger.infof("Execute %s", path);
     var sql = ResourceUtil.getResourceAsString(path);
     try (var connection = dataSource.getConnection()) {
       try (var statement = connection.createStatement()) {