From 36ba6dec2e1a1f367828c0832066836143c7e94e Mon Sep 17 00:00:00 2001
From: felmer <felmer>
Date: Wed, 15 Aug 2007 09:01:35 +0000
Subject: [PATCH] improve logging

SVN: 1372
---
 .../cisd/dbmigration/postgresql/PostgreSQLAdminDAO.java   | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/dbmigration/source/java/ch/systemsx/cisd/dbmigration/postgresql/PostgreSQLAdminDAO.java b/dbmigration/source/java/ch/systemsx/cisd/dbmigration/postgresql/PostgreSQLAdminDAO.java
index 4ae8800d11b..6ddc989d647 100644
--- a/dbmigration/source/java/ch/systemsx/cisd/dbmigration/postgresql/PostgreSQLAdminDAO.java
+++ b/dbmigration/source/java/ch/systemsx/cisd/dbmigration/postgresql/PostgreSQLAdminDAO.java
@@ -23,7 +23,6 @@ import org.springframework.dao.DataAccessException;
 import org.springframework.jdbc.core.JdbcTemplate;
 import org.springframework.jdbc.core.simple.SimpleJdbcDaoSupport;
 
-import ch.systemsx.cisd.common.db.SQLStateUtils;
 import ch.systemsx.cisd.common.logging.LogCategory;
 import ch.systemsx.cisd.common.logging.LogFactory;
 import ch.systemsx.cisd.dbmigration.DBUtilities;
@@ -88,22 +87,21 @@ public class PostgreSQLAdminDAO extends SimpleJdbcDaoSupport implements IDatabas
 
     public void createDatabase()
     {
+        operationLog.info("Try to create empty database '" + database + "' with owner '" + owner + "'.");
         JdbcTemplate jdbcTemplate = getJdbcTemplate();
         jdbcTemplate.execute("create database " + database + " with owner = " + owner
                                         + " encoding = 'utf8' tablespace = pg_default;" 
                              + "alter database " + database + " set default_with_oids = off;");
         try
         {
+            operationLog.info("Try to create language 'plpgsql'.");
             jdbcTemplate.execute("create trusted procedural language 'plpgsql' handler plpgsql_call_handler "
                                  + "validator plpgsql_validator;");
         } catch (DataAccessException e)
         {
             if (DBUtilities.isDuplicateObjectException(e))
             {
-                if (operationLog.isInfoEnabled())
-                {
-                    operationLog.info("Couldn't create language:" + e+":"+SQLStateUtils.getSqlState(e));
-                }
+                operationLog.info("Language 'plpgsql' already exists.");
             } else
             {
                 operationLog.error("Database language 'plpgsql' couldn't be created:", e);
-- 
GitLab