Github user franz1981 commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2181#discussion_r202040381
--- Diff:
artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/drivers/AbstractJDBCDriver.java
---
@@ -227,7 +227,11 @@ private void createTableIfNotExists(String tableName,
String... sqls) throws SQL
}
}
} catch (SQLException e) {
- logger.warn(JDBCUtils.appendSQLExceptionDetails(new
StringBuilder("Can't verify the initialization of table
").append(tableName).append(" due to:"), e,
sqlProvider.getCountJournalRecordsSQL()));
+ if (logger.isDebugEnabled()) {
+ logger.debug(JDBCUtils.appendSQLExceptionDetails(new
StringBuilder("Can't verify the initialization of table
").append(tableName).append(" due to:"), e,
sqlProvider.getCountJournalRecordsSQL()));
+ } else {
+ logger.infof("Can't verify the initialization of table
%s", tableName);
--- End diff --
which one? debug or infof or both?
I think it worths to know that something has happened at least...just don't
want to scare users for nothing...
---
