March 2010 - logback-dev - mailman3.qos.ch

[GIT] Logback: the generic, reliable, fast and flexible logging framework. branch, master, updated. v0.9.18-107-g901a3a1
by git-noreply＠pixie.qos.ch 22 Mar '10

22 Mar '10

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Logback: the generic, reliable, fast and flexible logging framework.". The branch, master has been updated via 901a3a19b1f89939a2fed00c2cdc5ebd8c8bec77 (commit) from 130277bce03f608db7ab555ec28705f9fc974e1e (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://git.qos.ch/gitweb/?p=logback.git;a=commit;h=901a3a19b1f89939a2fed00c… http://github.com/ceki/logback/commit/901a3a19b1f89939a2fed00c2cdc5ebd8c8be… commit 901a3a19b1f89939a2fed00c2cdc5ebd8c8bec77 Author: Ceki Gulcu <ceki(a)qos.ch> Date: Mon Mar 22 21:43:33 2010 +0100 - adding H2 and Sybase support diff --git a/logback-core/src/main/java/ch/qos/logback/core/db/dialect/DBUtil.java b/logback-core/src/main/java/ch/qos/logback/core/db/dialect/DBUtil.java index 330b573..cfb90c2 100644 --- a/logback-core/src/main/java/ch/qos/logback/core/db/dialect/DBUtil.java +++ b/logback-core/src/main/java/ch/qos/logback/core/db/dialect/DBUtil.java @@ -30,6 +30,8 @@ public class DBUtil extends ContextAwareBase { // private static final String MSSQL_PART = "mssqlserver4"; private static final String MSSQL_PART = "microsoft"; private static final String HSQL_PART = "hsql"; + private static final String H2_PART = "h2"; + private static final String SYBASE_SQLANY_PART = "sql anywhere"; public static SQLDialectCode discoverSQLDialect(DatabaseMetaData meta) { SQLDialectCode dialectCode = SQLDialectCode.UNKNOWN_DIALECT; @@ -48,6 +50,10 @@ public class DBUtil extends ContextAwareBase { return SQLDialectCode.MSSQL_DIALECT; } else if (dbName.indexOf(HSQL_PART) != -1) { return SQLDialectCode.HSQL_DIALECT; + } else if (dbName.indexOf(H2_PART) != -1) { + return SQLDialectCode.H2_DIALECT; + } else if (dbName.indexOf(SYBASE_SQLANY_PART) != -1) { + return SQLDialectCode.SYBASE_SQLANYWHERE_DIALECT; } else { return SQLDialectCode.UNKNOWN_DIALECT; } @@ -64,24 +70,31 @@ public class DBUtil extends ContextAwareBase { switch (sqlDialectType) { case POSTGRES_DIALECT: sqlDialect = new PostgreSQLDialect(); - break; + case MYSQL_DIALECT: sqlDialect = new MySQLDialect(); - break; + case ORACLE_DIALECT: sqlDialect = new OracleDialect(); - break; + case MSSQL_DIALECT: sqlDialect = new MsSQLDialect(); - break; + case HSQL_DIALECT: sqlDialect = new HSQLDBDialect(); + case H2_DIALECT: + sqlDialect = new H2Dialect(); + break; + + case SYBASE_SQLANYWHERE_DIALECT: + sqlDialect = new SybaseSqlAnywhereDialect(); break; + } return sqlDialect; } diff --git a/logback-core/src/main/java/ch/qos/logback/core/db/dialect/SQLDialectCode.java b/logback-core/src/main/java/ch/qos/logback/core/db/dialect/SQLDialectCode.java index 3176036..b643dec 100644 --- a/logback-core/src/main/java/ch/qos/logback/core/db/dialect/SQLDialectCode.java +++ b/logback-core/src/main/java/ch/qos/logback/core/db/dialect/SQLDialectCode.java @@ -19,5 +19,7 @@ public enum SQLDialectCode { MYSQL_DIALECT, ORACLE_DIALECT, MSSQL_DIALECT, - HSQL_DIALECT; + HSQL_DIALECT, + H2_DIALECT, + SYBASE_SQLANYWHERE_DIALECT; } diff --git a/logback-site/src/site/pages/news.html b/logback-site/src/site/pages/news.html index 664b15a..92ca237 100644 --- a/logback-site/src/site/pages/news.html +++ b/logback-site/src/site/pages/news.html @@ -134,17 +134,26 @@ reported by Tom SH Liu. - DBAppender now outputs exception class and message. It will also output root causes, fixing <a href="http://jira.qos.ch/browse/LBCLASSIC-170">LBCLASSIC-170</a> reported by Tomasz Nurkiewicz. + You can now override table and column names in DBAppender. This + feature was requested in <a + href="http://jira.qos.ch/browse/LBCLASSIC-188">LBCLASSIC-188</a> + reported by Tomasz Nurkiewicz. + + Fixed missing EVENT_ID column problem on PostgreSQL as reported in <a href="http://jira.qos.ch/browse/LBCORE-126">LBCORE-126</a> by Brian Edwards. + Added Sybase SQLAnywhere support in DBAppender. This feature + was requested by Michael Lynch in <a + href="http://jira.qos.ch/browse/LBCLASSIC-197">LBCLASSIC-197</a>. + Fixed <a href="http://jira.qos.ch/browse/LBSITE-36">LBSITE-36</a> reported by John Jimenez. ----------------------------------------------------------------------- Summary of changes: .../ch/qos/logback/core/db/dialect/DBUtil.java | 21 ++++++++++++++++--- .../logback/core/db/dialect/SQLDialectCode.java | 4 ++- logback-site/src/site/pages/news.html | 11 +++++++++- 3 files changed, 30 insertions(+), 6 deletions(-) hooks/post-receive -- Logback: the generic, reliable, fast and flexible logging framework.

1 0

[GIT] Logback: the generic, reliable, fast and flexible logging framework. branch, master, updated. v0.9.18-106-g130277b
by git-noreply＠pixie.qos.ch 22 Mar '10

22 Mar '10

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Logback: the generic, reliable, fast and flexible logging framework.". The branch, master has been updated via 130277bce03f608db7ab555ec28705f9fc974e1e (commit) from 39dbbf9a5f7d414db63e89b18d09d04fd0ae6498 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://git.qos.ch/gitweb/?p=logback.git;a=commit;h=130277bce03f608db7ab555e… http://github.com/ceki/logback/commit/130277bce03f608db7ab555ec28705f9fc974… commit 130277bce03f608db7ab555ec28705f9fc974e1e Author: Ceki Gulcu <ceki(a)qos.ch> Date: Mon Mar 22 17:46:26 2010 +0100 Fixed LBCLASSIC-170 diff --git a/logback-classic/src/main/java/ch/qos/logback/classic/db/DBAppender.java b/logback-classic/src/main/java/ch/qos/logback/classic/db/DBAppender.java index e563109..7c92e50 100644 --- a/logback-classic/src/main/java/ch/qos/logback/classic/db/DBAppender.java +++ b/logback-classic/src/main/java/ch/qos/logback/classic/db/DBAppender.java @@ -25,15 +25,18 @@ import java.util.Set; import ch.qos.logback.classic.db.names.DBNameResolver; import ch.qos.logback.classic.db.names.DefaultDBNameResolver; import ch.qos.logback.classic.spi.ILoggingEvent; +import ch.qos.logback.classic.spi.IThrowableProxy; import ch.qos.logback.classic.spi.StackTraceElementProxy; +import ch.qos.logback.classic.spi.ThrowableProxyUtil; +import ch.qos.logback.core.CoreConstants; import ch.qos.logback.core.db.DBAppenderBase; /** * The DBAppender inserts logging events into three database tables in a format - * independent of the Java programming language. + * independent of the Java programming language. * - * For more information about this appender, please refer to the online manual at - * http://logback.qos.ch/manual/appenders.html#DBAppender + * For more information about this appender, please refer to the online manual + * at http://logback.qos.ch/manual/appenders.html#DBAppender * * @author Ceki Gülcü * @author Ray DeCampo @@ -51,7 +54,7 @@ public class DBAppender extends DBAppenderBase<ILoggingEvent> { // PreparedStatement.getGeneratedKeys() method was added in JDK 1.4 Method getGeneratedKeysMethod; try { - // the + // the getGeneratedKeysMethod = PreparedStatement.class.getMethod( "getGeneratedKeys", (Class[]) null); } catch (Exception ex) { @@ -59,7 +62,7 @@ public class DBAppender extends DBAppenderBase<ILoggingEvent> { } GET_GENERATED_KEYS_METHOD = getGeneratedKeysMethod; } - + public DBAppender() { } @@ -69,7 +72,7 @@ public class DBAppender extends DBAppenderBase<ILoggingEvent> { @Override public void start() { - if(dbNameResolver == null) + if (dbNameResolver == null) dbNameResolver = new DefaultDBNameResolver(); insertExceptionSQL = SQLBuilder.buildInsertExceptionSQL(dbNameResolver); insertPropertiesSQL = SQLBuilder.buildInsertPropertiesSQL(dbNameResolver); @@ -85,7 +88,7 @@ public class DBAppender extends DBAppenderBase<ILoggingEvent> { bindLoggingEventWithInsertStatement(insertStatement, event); // This is expensive... should we do it every time? bindCallerDataWithPreparedStatement(insertStatement, event.getCallerData()); - + int updateCount = insertStatement.executeUpdate(); if (updateCount != 1) { addWarn("Failed to insert loggingEvent"); @@ -97,12 +100,12 @@ public class DBAppender extends DBAppenderBase<ILoggingEvent> { insertProperties(mergedMap, connection, eventId); if (event.getThrowableProxy() != null) { - insertThrowable(event.getThrowableProxy().getStackTraceElementProxyArray(), connection, eventId); + insertThrowable(event.getThrowableProxy(), connection, eventId); } } - void bindLoggingEventWithInsertStatement(PreparedStatement stmt, ILoggingEvent event) - throws SQLException { + void bindLoggingEventWithInsertStatement(PreparedStatement stmt, + ILoggingEvent event) throws SQLException { stmt.setLong(1, event.getTimeStamp()); stmt.setString(2, event.getFormattedMessage()); stmt.setString(3, event.getLoggerName()); @@ -111,8 +114,8 @@ public class DBAppender extends DBAppenderBase<ILoggingEvent> { stmt.setShort(6, DBHelper.computeReferenceMask(event)); } - void bindCallerDataWithPreparedStatement(PreparedStatement stmt, StackTraceElement[] callerDataArray) - throws SQLException { + void bindCallerDataWithPreparedStatement(PreparedStatement stmt, + StackTraceElement[] callerDataArray) throws SQLException { StackTraceElement callerData = callerDataArray[0]; if (callerData != null) { stmt.setString(7, callerData.getFileName()); @@ -128,7 +131,8 @@ public class DBAppender extends DBAppenderBase<ILoggingEvent> { // we consider that event-specific properties should have priority over // context-wide // properties. - Map<String, String> loggerContextMap = event.getLoggerContextVO().getPropertyMap(); + Map<String, String> loggerContextMap = event.getLoggerContextVO() + .getPropertyMap(); Map<String, String> mdcMap = event.getMDCPropertyMap(); if (loggerContextMap != null) { mergedMap.putAll(loggerContextMap); @@ -149,7 +153,7 @@ public class DBAppender extends DBAppenderBase<ILoggingEvent> { protected String getInsertSQL() { return insertSQL; } - + protected void insertProperties(Map<String, String> mergedMap, Connection connection, int eventId) throws SQLException { Set propertiesKeys = mergedMap.keySet(); @@ -180,28 +184,71 @@ public class DBAppender extends DBAppenderBase<ILoggingEvent> { insertPropertiesStatement = null; } } - - protected void insertThrowable(StackTraceElementProxy[] stepArray, Connection connection, + + /** + * Add an exception statement either as a batch or execute immediately if + * batch updates are not supported. + */ + void updateExceptionStatement(PreparedStatement exceptionStatement, + String txt, short i, int eventId) throws SQLException { + exceptionStatement.setInt(1, eventId); + exceptionStatement.setShort(2, i); + exceptionStatement.setString(3, txt); + if (cnxSupportsBatchUpdates) { + exceptionStatement.addBatch(); + } else { + exceptionStatement.execute(); + } + } + + short buildExceptionStatement(IThrowableProxy tp, short baseIndex, + PreparedStatement insertExceptionStatement, int eventId) + throws SQLException { + + StringBuilder buf = new StringBuilder(); + ThrowableProxyUtil.printFirstLine(buf, tp); + updateExceptionStatement(insertExceptionStatement, buf.toString(), + baseIndex++, eventId); + + int commonFrames = tp.getCommonFrames(); + StackTraceElementProxy[] stepArray = tp.getStackTraceElementProxyArray(); + for (int i = 0; i < stepArray.length - commonFrames; i++) { + StringBuilder sb = new StringBuilder(); + sb.append(CoreConstants.TAB); + ThrowableProxyUtil.printSTEP(sb, stepArray[i]); + updateExceptionStatement(insertExceptionStatement, sb.toString(), + baseIndex++, eventId); + } + + if (commonFrames > 0) { + StringBuilder sb = new StringBuilder(); + sb.append(CoreConstants.TAB).append("... ").append(commonFrames).append( + " common frames omitted"); + updateExceptionStatement(insertExceptionStatement, sb.toString(), + baseIndex++, eventId); + } + + return baseIndex; + } + + protected void insertThrowable(IThrowableProxy tp, Connection connection, int eventId) throws SQLException { - PreparedStatement insertExceptionStatement = connection + PreparedStatement exceptionStatement = connection .prepareStatement(insertExceptionSQL); - for (short i = 0; i < stepArray.length; i++) { - insertExceptionStatement.setInt(1, eventId); - insertExceptionStatement.setShort(2, i); - insertExceptionStatement.setString(3, stepArray[i].toString()); - if (cnxSupportsBatchUpdates) { - insertExceptionStatement.addBatch(); - } else { - insertExceptionStatement.execute(); - } + short baseIndex = 0; + while (tp != null) { + baseIndex = buildExceptionStatement(tp, baseIndex, exceptionStatement, + eventId); + tp = tp.getCause(); } + if (cnxSupportsBatchUpdates) { - insertExceptionStatement.executeBatch(); + exceptionStatement.executeBatch(); } - insertExceptionStatement.close(); - insertExceptionStatement = null; + exceptionStatement.close(); + exceptionStatement = null; } } diff --git a/logback-classic/src/main/java/ch/qos/logback/classic/pattern/ExtendedThrowableProxyConverter.java b/logback-classic/src/main/java/ch/qos/logback/classic/pattern/ExtendedThrowableProxyConverter.java index a7e545c..4a841d7 100644 --- a/logback-classic/src/main/java/ch/qos/logback/classic/pattern/ExtendedThrowableProxyConverter.java +++ b/logback-classic/src/main/java/ch/qos/logback/classic/pattern/ExtendedThrowableProxyConverter.java @@ -13,21 +13,16 @@ */ package ch.qos.logback.classic.pattern; -import ch.qos.logback.classic.spi.ClassPackagingData; import ch.qos.logback.classic.spi.ILoggingEvent; import ch.qos.logback.classic.spi.StackTraceElementProxy; +import ch.qos.logback.classic.spi.ThrowableProxyUtil; public class ExtendedThrowableProxyConverter extends ThrowableProxyConverter { @Override protected void extraData(StringBuilder builder, StackTraceElementProxy step) { - if (step != null) { - ClassPackagingData pi = step.getClassPackagingData(); - if (pi != null) { - builder.append(" [").append(pi.getCodeLocation()).append(':').append( - pi.getVersion()).append(']'); - } + ThrowableProxyUtil.appendPackagingData(builder, step); } } diff --git a/logback-classic/src/main/java/ch/qos/logback/classic/spi/ThrowableProxyUtil.java b/logback-classic/src/main/java/ch/qos/logback/classic/spi/ThrowableProxyUtil.java index 5b45359..be8dc67 100644 --- a/logback-classic/src/main/java/ch/qos/logback/classic/spi/ThrowableProxyUtil.java +++ b/logback-classic/src/main/java/ch/qos/logback/classic/spi/ThrowableProxyUtil.java @@ -81,7 +81,7 @@ public class ThrowableProxyUtil { return sb.toString(); } - static void appendPackagingData(StringBuilder builder, StackTraceElementProxy step) { + public static void appendPackagingData(StringBuilder builder, StackTraceElementProxy step) { if (step != null) { ClassPackagingData cpd = step.getClassPackagingData(); if (cpd != null) { diff --git a/logback-classic/src/test/java/ch/qos/logback/classic/db/DBAppenderIntegrationTest.java b/logback-classic/src/test/java/ch/qos/logback/classic/db/DBAppenderIntegrationTest.java index 5d1f26b..e146669 100644 --- a/logback-classic/src/test/java/ch/qos/logback/classic/db/DBAppenderIntegrationTest.java +++ b/logback-classic/src/test/java/ch/qos/logback/classic/db/DBAppenderIntegrationTest.java @@ -76,14 +76,21 @@ public class DBAppenderIntegrationTest { for (int i = 1; i <= runLength; i++) { logger.debug("This is a debug message. Message number: " + i); } - logger.error("At last an error.", new Exception("Just testing")); + Exception e = new Exception("Just testing", getCause()); + logger + .error("At last an error.", e); + e.printStackTrace(); // check that there were no errors StatusPrinter.print(lc); assertEquals(Status.INFO, lc.getStatusManager().getLevel()); } + Throwable getCause() { + return new IllegalStateException("test cause"); + } + static boolean isConformingHostAndJDK16OrHigher(String[] conformingHostList) { if (!Env.isJDK6OrHigher()) { return false; diff --git a/logback-site/src/site/pages/news.html b/logback-site/src/site/pages/news.html index 912666c..664b15a 100644 --- a/logback-site/src/site/pages/news.html +++ b/logback-site/src/site/pages/news.html @@ -134,6 +134,17 @@ reported by Tom SH Liu. + + DBAppender now outputs exception class and message. It will + also output root causes, fixing + <a href="http://jira.qos.ch/browse/LBCLASSIC-170">LBCLASSIC-170</a> + reported by Tomasz Nurkiewicz. + + + Fixed missing EVENT_ID column problem on PostgreSQL as reported + in <a href="http://jira.qos.ch/browse/LBCORE-126">LBCORE-126</a> + by Brian Edwards. + Fixed <a href="http://jira.qos.ch/browse/LBSITE-36">LBSITE-36</a> reported by John Jimenez. ----------------------------------------------------------------------- Summary of changes: .../java/ch/qos/logback/classic/db/DBAppender.java | 105 ++++++++++++++------ .../pattern/ExtendedThrowableProxyConverter.java | 9 +-- .../logback/classic/spi/ThrowableProxyUtil.java | 2 +- .../classic/db/DBAppenderIntegrationTest.java | 9 ++- logback-site/src/site/pages/news.html | 11 ++ 5 files changed, 98 insertions(+), 38 deletions(-) hooks/post-receive -- Logback: the generic, reliable, fast and flexible logging framework.

1 0

[JIRA] Created: (LBCORE-126) ch.qos.logback.core.db.DBAppenderBase tries to use column "EVENT_ID" but does not exist
by Brian Edwards (JIRA) 22 Mar '10

22 Mar '10

ch.qos.logback.core.db.DBAppenderBase tries to use column "EVENT_ID" but does not exist --------------------------------------------------------------------------------------- Key: LBCORE-126 URL: http://jira.qos.ch/browse/LBCORE-126 Project: logback-core Issue Type: Bug Components: Appender Affects Versions: 0.9.17 Environment: postgresql Reporter: Brian Edwards Assignee: Logback dev list on line 91 in ch.qos.logback.core.db.DBAppenderBase append creates a java.sql.PreparedStatement by calling connection.prepareStatement(getInsertSQL(), new String[] {"EVENT_ID"}); but because postgres is case sensitive the column does not exist because it is called event_id -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.qos.ch/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

2 2

[GIT] Logback: the generic, reliable, fast and flexible logging framework. branch, master, updated. v0.9.18-105-g39dbbf9
by git-noreply＠pixie.qos.ch 22 Mar '10

22 Mar '10

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Logback: the generic, reliable, fast and flexible logging framework.". The branch, master has been updated via 39dbbf9a5f7d414db63e89b18d09d04fd0ae6498 (commit) from d3a98f905ea1bffb7afda7fb4405c28238635a57 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://git.qos.ch/gitweb/?p=logback.git;a=commit;h=39dbbf9a5f7d414db63e89b1… http://github.com/ceki/logback/commit/39dbbf9a5f7d414db63e89b18d09d04fd0ae6… commit 39dbbf9a5f7d414db63e89b18d09d04fd0ae6498 Author: Ceki Gulcu <ceki(a)qos.ch> Date: Mon Mar 22 15:37:28 2010 +0100 - The virtual host 'gimmel' got corrupted and was resuscitated as 'haro'. FileAppenderResilience_AS_ROOT_Test and postgresql() in DBAppenderIntegrationTest now run on 'haro'. - Fixed LBCORE-126 by lowercasing the EVENT_ID column name in prepared statements for PostgreSQL. - corrected a few compiler warnings for "raw types" - updated to PostgreSQL JDBC driver to 8.4-701.jdbc4 diff --git a/logback-classic/pom.xml b/logback-classic/pom.xml index bdb2ec6..41559e0 100644 --- a/logback-classic/pom.xml +++ b/logback-classic/pom.xml @@ -82,6 +82,12 @@ </dependency> <dependency> + <groupId>postgresql</groupId> + <artifactId>postgresql</artifactId> + <scope>test</scope> + </dependency> + + <dependency> <groupId>javax.mail</groupId> <artifactId>mail</artifactId> <scope>compile</scope> @@ -304,12 +310,6 @@ org.slf4j.test_osgi <scope>test</scope> </dependency> - <dependency> - <groupId>postgresql</groupId> - <artifactId>postgresql</artifactId> - <version>8.2-507.jdbc3</version> - </dependency> -  <dependency> <groupId>com.microsoft.sqlserver</groupId> @@ -330,4 +330,4 @@ org.slf4j.test_osgi </profiles> -</project> \ No newline at end of file +</project> diff --git a/logback-classic/src/main/java/ch/qos/logback/classic/db/DBAppender.java b/logback-classic/src/main/java/ch/qos/logback/classic/db/DBAppender.java index e055c5b..e563109 100644 --- a/logback-classic/src/main/java/ch/qos/logback/classic/db/DBAppender.java +++ b/logback-classic/src/main/java/ch/qos/logback/classic/db/DBAppender.java @@ -85,7 +85,7 @@ public class DBAppender extends DBAppenderBase<ILoggingEvent> { bindLoggingEventWithInsertStatement(insertStatement, event); // This is expensive... should we do it every time? bindCallerDataWithPreparedStatement(insertStatement, event.getCallerData()); - + int updateCount = insertStatement.executeUpdate(); if (updateCount != 1) { addWarn("Failed to insert loggingEvent"); diff --git a/logback-classic/src/main/java/ch/qos/logback/classic/db/dialect/postgresql.sql b/logback-classic/src/main/java/ch/qos/logback/classic/db/dialect/postgresql.sql index 3f2ce1b..55c7343 100644 --- a/logback-classic/src/main/java/ch/qos/logback/classic/db/dialect/postgresql.sql +++ b/logback-classic/src/main/java/ch/qos/logback/classic/db/dialect/postgresql.sql @@ -1,11 +1,11 @@ -# This SQL script creates the required tables by ch.qos.logback.classic.db.DBAppender -# -# It is intended for PostgreSQL databases. +-- This SQL script creates the required tables by ch.qos.logback.classic.db.DBAppender +-- +-- It is intended for PostgreSQL databases. DROP TABLE logging_event_property; DROP TABLE logging_event_exception; -DROP SEQUENCE logging_event_id_seq; DROP TABLE logging_event; +DROP SEQUENCE logging_event_id_seq; CREATE SEQUENCE logging_event_id_seq MINVALUE 1 START 1; diff --git a/logback-classic/src/test/input/integration/db/postgresql-with-driver.xml b/logback-classic/src/test/input/integration/db/postgresql-with-driver.xml index ce5a3cf..976ddc7 100644 --- a/logback-classic/src/test/input/integration/db/postgresql-with-driver.xml +++ b/logback-classic/src/test/input/integration/db/postgresql-with-driver.xml @@ -2,17 +2,17 @@ <configuration> - <appender name="DB" class="ch.qos.logback.classic.db.DBAppender"> - <connectionSource class="ch.qos.logback.core.db.DriverManagerConnectionSource"> - <driverClass>org.postgresql.Driver</driverClass> - <url>jdbc:postgresql://192.168.1.5:5432/test</url> - <user>logback</user> - <password>logback</password> - </connectionSource> - </appender> - - <root> - <level value="debug" /> - <appender-ref ref="DB" /> - </root> + <appender name="DB" class="ch.qos.logback.classic.db.DBAppender"> + <connectionSource class="ch.qos.logback.core.db.DriverManagerConnectionSource"> + <driverClass>org.postgresql.Driver</driverClass> + <url>jdbc:postgresql://localhost:5432/test</url> + <user>test</user> + <password>test</password> + </connectionSource> + </appender> + + <root> + <level value="debug" /> + <appender-ref ref="DB" /> + </root> </configuration> diff --git a/logback-classic/src/test/java/ch/qos/logback/classic/PatternLayoutTest.java b/logback-classic/src/test/java/ch/qos/logback/classic/PatternLayoutTest.java index e2852b9..8ede4bc 100644 --- a/logback-classic/src/test/java/ch/qos/logback/classic/PatternLayoutTest.java +++ b/logback-classic/src/test/java/ch/qos/logback/classic/PatternLayoutTest.java @@ -36,7 +36,7 @@ import ch.qos.logback.core.pattern.PatternLayoutBase; import ch.qos.logback.core.pattern.parser.AbstractPatternLayoutBaseTest; import ch.qos.logback.core.testUtil.StringListAppender; -public class PatternLayoutTest extends AbstractPatternLayoutBaseTest { +public class PatternLayoutTest extends AbstractPatternLayoutBaseTest<ILoggingEvent> { private PatternLayout pl = new PatternLayout(); private LoggerContext lc = new LoggerContext(); @@ -44,7 +44,7 @@ public class PatternLayoutTest extends AbstractPatternLayoutBaseTest { Logger root = lc.getLogger(Logger.ROOT_LOGGER_NAME); ILoggingEvent le; - List optionList = new ArrayList(); + List<String> optionList = new ArrayList<String>(); public PatternLayoutTest() { super(); @@ -68,7 +68,7 @@ public class PatternLayoutTest extends AbstractPatternLayoutBaseTest { return makeLoggingEvent(null); } - public PatternLayoutBase getPatternLayoutBase() { + public PatternLayoutBase<ILoggingEvent> getPatternLayoutBase() { return new PatternLayout(); } @@ -170,7 +170,7 @@ public class PatternLayoutTest extends AbstractPatternLayoutBaseTest { root.getAppender("LIST"); String msg = "Simon says"; logger.debug(msg); - StringListAppender sla = (StringListAppender) root.getAppender("LIST"); + StringListAppender<ILoggingEvent> sla = (StringListAppender<ILoggingEvent>) root.getAppender("LIST"); assertNotNull(sla); assertEquals(1, sla.strList.size()); assertEquals(SampleConverter.SAMPLE_STR+" - "+msg, sla.strList.get(0)); diff --git a/logback-classic/src/test/java/ch/qos/logback/classic/db/DBAppenderIntegrationTest.java b/logback-classic/src/test/java/ch/qos/logback/classic/db/DBAppenderIntegrationTest.java index f603972..5d1f26b 100644 --- a/logback-classic/src/test/java/ch/qos/logback/classic/db/DBAppenderIntegrationTest.java +++ b/logback-classic/src/test/java/ch/qos/logback/classic/db/DBAppenderIntegrationTest.java @@ -38,10 +38,11 @@ public class DBAppenderIntegrationTest { static String LOCAL_HOST_NAME; static String[] CONFORMING_HOST_LIST = new String[] { "Orion" }; + static String[] POSTGRES_CONFORMING_HOST_LIST = new String[] { "haro" }; int diff = new Random(System.nanoTime()).nextInt(10000); LoggerContext lc = new LoggerContext(); - + @BeforeClass public static void setUpBeforeClass() throws Exception { InetAddress localhostIA = InetAddress.getLocalHost(); @@ -54,7 +55,7 @@ public class DBAppenderIntegrationTest { @Before public void setUp() throws Exception { - lc.setName("lc"+diff); + lc.setName("lc" + diff); } @After @@ -76,18 +77,18 @@ public class DBAppenderIntegrationTest { logger.debug("This is a debug message. Message number: " + i); } logger.error("At last an error.", new Exception("Just testing")); - + // check that there were no errors StatusPrinter.print(lc); assertEquals(Status.INFO, lc.getStatusManager().getLevel()); - + } - - static boolean isConformingHostAndJDK16OrHigher() { - if(!Env.isJDK6OrHigher()) { + + static boolean isConformingHostAndJDK16OrHigher(String[] conformingHostList) { + if (!Env.isJDK6OrHigher()) { return false; } - for (String conformingHost : CONFORMING_HOST_LIST) { + for (String conformingHost : conformingHostList) { if (conformingHost.equalsIgnoreCase(LOCAL_HOST_NAME)) { return true; } @@ -95,6 +96,10 @@ public class DBAppenderIntegrationTest { return false; } + static boolean isConformingHostAndJDK16OrHigher() { + return isConformingHostAndJDK16OrHigher(CONFORMING_HOST_LIST); + } + @Test public void sqlserver() throws Exception { // perform test only on conforming hosts @@ -123,7 +128,7 @@ public class DBAppenderIntegrationTest { } doTest("src/test/input/integration/db/oracle11g-with-driver.xml"); } - + @Test public void mysql() throws Exception { // perform test only on conforming hosts @@ -132,14 +137,15 @@ public class DBAppenderIntegrationTest { } doTest("src/test/input/integration/db/mysql-with-driver.xml"); } - + @Test public void postgres() throws Exception { // perform test only on conforming hosts - if (!isConformingHostAndJDK16OrHigher()) { + if (!isConformingHostAndJDK16OrHigher(POSTGRES_CONFORMING_HOST_LIST)) { return; } + System.out.println("doing test"); doTest("src/test/input/integration/db/postgresql-with-driver.xml"); } - + } diff --git a/logback-classic/src/test/java/ch/qos/logback/classic/util/InitializationTest.java b/logback-classic/src/test/java/ch/qos/logback/classic/util/InitializationTest.java index cfd81ca..519c1eb 100644 --- a/logback-classic/src/test/java/ch/qos/logback/classic/util/InitializationTest.java +++ b/logback-classic/src/test/java/ch/qos/logback/classic/util/InitializationTest.java @@ -30,6 +30,7 @@ import org.slf4j.LoggerFactory; import ch.qos.logback.classic.ClassicTestConstants; import ch.qos.logback.classic.Logger; import ch.qos.logback.classic.LoggerContext; +import ch.qos.logback.classic.spi.ILoggingEvent; import ch.qos.logback.core.Appender; import ch.qos.logback.core.ConsoleAppender; import ch.qos.logback.core.joran.spi.JoranException; @@ -60,7 +61,7 @@ public class InitializationTest { // However, this is something we try to avoid in order to simplify the life // of users trying to follows the manual and logback-examples from an IDE public void atoconfig() { - Appender appender = root.getAppender("STDOUT"); + Appender<ILoggingEvent> appender = root.getAppender("STDOUT"); assertNotNull(appender); assertTrue(appender instanceof ConsoleAppender); } @@ -79,7 +80,7 @@ public class InitializationTest { } { lc.stop(); - Appender appender = root.getAppender("STDOUT"); + Appender<ILoggingEvent> appender = root.getAppender("STDOUT"); assertNull(appender); } } @@ -96,7 +97,7 @@ public class InitializationTest { //lc.reset(); System.setProperty(ContextInitializer.CONFIG_FILE_PROPERTY, val); new ContextInitializer(lc).autoConfig(); - Appender appender = root.getAppender("AUTO_BY_SYSTEM_PROPERTY"); + Appender<ILoggingEvent> appender = root.getAppender("AUTO_BY_SYSTEM_PROPERTY"); assertNotNull(appender); } diff --git a/logback-core/src/main/java/ch/qos/logback/core/db/DBAppenderBase.java b/logback-core/src/main/java/ch/qos/logback/core/db/DBAppenderBase.java index bcac3ef..be8bc33 100644 --- a/logback-core/src/main/java/ch/qos/logback/core/db/DBAppenderBase.java +++ b/logback-core/src/main/java/ch/qos/logback/core/db/DBAppenderBase.java @@ -24,6 +24,7 @@ import java.sql.Statement; import ch.qos.logback.core.AppenderBase; import ch.qos.logback.core.db.dialect.DBUtil; import ch.qos.logback.core.db.dialect.SQLDialect; +import ch.qos.logback.core.db.dialect.SQLDialectCode; /** * @author Ceki Gülcü @@ -82,6 +83,7 @@ public abstract class DBAppenderBase<E> extends AppenderBase<E> { this.connectionSource = connectionSource; } + @Override public void append(E eventObject) { Connection connection = null; @@ -90,8 +92,13 @@ public abstract class DBAppenderBase<E> extends AppenderBase<E> { connection.setAutoCommit(false); PreparedStatement insertStatement; if (cnxSupportsGetGeneratedKeys) { + String EVENT_ID_COL_NAME = "EVENT_ID"; + // see + if(connectionSource.getSQLDialectCode() == SQLDialectCode.POSTGRES_DIALECT) { + EVENT_ID_COL_NAME = EVENT_ID_COL_NAME.toLowerCase(); + } insertStatement = connection.prepareStatement(getInsertSQL(), - new String[] {"EVENT_ID"}); + new String[] {EVENT_ID_COL_NAME}); } else { insertStatement = connection.prepareStatement(getInsertSQL()); } diff --git a/logback-core/src/main/java/ch/qos/logback/core/sift/AppenderTrackerImpl.java b/logback-core/src/main/java/ch/qos/logback/core/sift/AppenderTrackerImpl.java index 4697b55..ca86f4c 100644 --- a/logback-core/src/main/java/ch/qos/logback/core/sift/AppenderTrackerImpl.java +++ b/logback-core/src/main/java/ch/qos/logback/core/sift/AppenderTrackerImpl.java @@ -91,7 +91,7 @@ public class AppenderTrackerImpl<E> implements AppenderTracker<E> { if(found != null) { rearrangePreexistingLinks(e); map.remove(key); - Appender appender = e.value; + Appender<E> appender = e.value; appender.stop(); } } diff --git a/logback-core/src/test/java/ch/qos/logback/core/FileAppenderResilience_AS_ROOT_Test.java b/logback-core/src/test/java/ch/qos/logback/core/FileAppenderResilience_AS_ROOT_Test.java index a5a7691..2e39ae6 100644 --- a/logback-core/src/test/java/ch/qos/logback/core/FileAppenderResilience_AS_ROOT_Test.java +++ b/logback-core/src/test/java/ch/qos/logback/core/FileAppenderResilience_AS_ROOT_Test.java @@ -47,7 +47,7 @@ public class FileAppenderResilience_AS_ROOT_Test { FileAppender<Object> fa = new FileAppender<Object>(); static boolean isConformingHost() { - return Env.isLocalHostNameInList(new String[] { "gimmel" }); + return Env.isLocalHostNameInList(new String[] { "haro" }); } @Before diff --git a/logback-core/src/test/java/ch/qos/logback/core/pattern/parser/SamplePatternLayoutTest.java b/logback-core/src/test/java/ch/qos/logback/core/pattern/parser/SamplePatternLayoutTest.java index 8bed2f4..edd5466 100644 --- a/logback-core/src/test/java/ch/qos/logback/core/pattern/parser/SamplePatternLayoutTest.java +++ b/logback-core/src/test/java/ch/qos/logback/core/pattern/parser/SamplePatternLayoutTest.java @@ -22,7 +22,7 @@ import ch.qos.logback.core.ContextBase; import ch.qos.logback.core.pattern.PatternLayoutBase; -public class SamplePatternLayoutTest extends AbstractPatternLayoutBaseTest { +public class SamplePatternLayoutTest extends AbstractPatternLayoutBaseTest<Object> { Context context = new ContextBase(); diff --git a/pom.xml b/pom.xml index d083acf..7f1e6ea 100755 --- a/pom.xml +++ b/pom.xml @@ -93,12 +93,11 @@ <artifactId>hsqldb</artifactId> <version>1.8.0.7</version> </dependency> - <dependency> - <groupId>com.h2database</groupId> - <artifactId>h2</artifactId> - <version>1.2.131</version> - </dependency> + <groupId>postgresql</groupId> + <artifactId>postgresql</artifactId> + <version>8.4-701.jdbc4</version> + </dependency> <dependency> <groupId>org.apache.tomcat</groupId> <artifactId>catalina</artifactId> @@ -125,10 +124,10 @@ <version>1.0</version> </dependency> <dependency> - <groupId>javax.servlet</groupId> - <artifactId>servlet-api</artifactId> - <version>2.5</version> - </dependency> + <groupId>javax.servlet</groupId> + <artifactId>servlet-api</artifactId> + <version>2.5</version> + </dependency> </dependencies> </dependencyManagement> @@ -362,4 +361,4 @@ </profiles> -</project> \ No newline at end of file +</project> ----------------------------------------------------------------------- Summary of changes: logback-classic/pom.xml | 14 ++++---- .../java/ch/qos/logback/classic/db/DBAppender.java | 2 +- .../qos/logback/classic/db/dialect/postgresql.sql | 8 ++-- .../integration/db/postgresql-with-driver.xml | 26 ++++++++-------- .../ch/qos/logback/classic/PatternLayoutTest.java | 8 ++-- .../classic/db/DBAppenderIntegrationTest.java | 30 ++++++++++++-------- .../logback/classic/util/InitializationTest.java | 7 ++-- .../ch/qos/logback/core/db/DBAppenderBase.java | 9 +++++- .../qos/logback/core/sift/AppenderTrackerImpl.java | 2 +- .../core/FileAppenderResilience_AS_ROOT_Test.java | 2 +- .../pattern/parser/SamplePatternLayoutTest.java | 2 +- pom.xml | 19 ++++++------ 12 files changed, 71 insertions(+), 58 deletions(-) hooks/post-receive -- Logback: the generic, reliable, fast and flexible logging framework.

1 0

[GIT] Logback: the generic, reliable, fast and flexible logging framework. branch, master, updated. v0.9.18-104-gd3a98f9
by git-noreply＠pixie.qos.ch 22 Mar '10

22 Mar '10

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Logback: the generic, reliable, fast and flexible logging framework.". The branch, master has been updated via d3a98f905ea1bffb7afda7fb4405c28238635a57 (commit) from 66fb7325add00aea9f037fc9e813f07df4648518 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://git.qos.ch/gitweb/?p=logback.git;a=commit;h=d3a98f905ea1bffb7afda7fb… http://github.com/ceki/logback/commit/d3a98f905ea1bffb7afda7fb4405c28238635… commit d3a98f905ea1bffb7afda7fb4405c28238635a57 Author: Ceki Gulcu <ceki(a)qos.ch> Date: Mon Mar 22 15:05:52 2010 +0100 - added req. MDCInsertingServletFilter - minor refactoring in DBAppenderBase diff --git a/logback-classic/src/main/java/ch/qos/logback/classic/ClassicConstants.java b/logback-classic/src/main/java/ch/qos/logback/classic/ClassicConstants.java index 12fedc5..8958016 100644 --- a/logback-classic/src/main/java/ch/qos/logback/classic/ClassicConstants.java +++ b/logback-classic/src/main/java/ch/qos/logback/classic/ClassicConstants.java @@ -36,6 +36,7 @@ public class ClassicConstants { public final static String REQUEST_REMOTE_HOST_MDC_KEY = "req.remoteHost"; public final static String REQUEST_USER_AGENT_MDC_KEY = "req.userAgent"; public final static String REQUEST_REQUEST_URI = "req.requestURI"; + public final static String REQUEST_QUERY_STRING = "req.queryString"; public final static String REQUEST_REQUEST_URL = "req.requestURL"; public final static String REQUEST_X_FORWARDED_FOR = "req.xForwardedFor"; diff --git a/logback-classic/src/main/java/ch/qos/logback/classic/helpers/MDCInsertingServletFilter.java b/logback-classic/src/main/java/ch/qos/logback/classic/helpers/MDCInsertingServletFilter.java index 5efae36..ce2d74f 100644 --- a/logback-classic/src/main/java/ch/qos/logback/classic/helpers/MDCInsertingServletFilter.java +++ b/logback-classic/src/main/java/ch/qos/logback/classic/helpers/MDCInsertingServletFilter.java @@ -67,6 +67,7 @@ public class MDCInsertingServletFilter implements Filter { if (requestURL != null) { MDC.put(ClassicConstants.REQUEST_REQUEST_URL, requestURL.toString()); } + MDC.put(ClassicConstants.REQUEST_QUERY_STRING, httpServletRequest.getQueryString()); MDC.put(ClassicConstants.REQUEST_USER_AGENT_MDC_KEY, httpServletRequest .getHeader("User-Agent")); MDC.put(ClassicConstants.REQUEST_X_FORWARDED_FOR, httpServletRequest @@ -78,6 +79,7 @@ public class MDCInsertingServletFilter implements Filter { void clearMDC() { MDC.remove(ClassicConstants.REQUEST_REMOTE_HOST_MDC_KEY); MDC.remove(ClassicConstants.REQUEST_REQUEST_URI); + MDC.remove(ClassicConstants.REQUEST_QUERY_STRING); // removing possibly inexistent item is OK MDC.remove(ClassicConstants.REQUEST_REQUEST_URL); MDC.remove(ClassicConstants.REQUEST_USER_AGENT_MDC_KEY); diff --git a/logback-core/src/main/java/ch/qos/logback/core/db/DBAppenderBase.java b/logback-core/src/main/java/ch/qos/logback/core/db/DBAppenderBase.java index bee265d..bcac3ef 100644 --- a/logback-core/src/main/java/ch/qos/logback/core/db/DBAppenderBase.java +++ b/logback-core/src/main/java/ch/qos/logback/core/db/DBAppenderBase.java @@ -98,10 +98,9 @@ public abstract class DBAppenderBase<E> extends AppenderBase<E> { subAppend(eventObject, connection, insertStatement); // we no longer need the insertStatement - if (insertStatement != null) { - insertStatement.close(); - insertStatement = null; - } + close(insertStatement); + insertStatement = null; + connection.commit(); } catch (Throwable sqle) { addError("problem appending event", sqle); @@ -137,9 +136,6 @@ public abstract class DBAppenderBase<E> extends AppenderBase<E> { } if (!gotGeneratedKeys) { - insertStatement.close(); - insertStatement = null; - idStatement = connection.createStatement(); idStatement.setMaxRows(1); rs = idStatement.executeQuery(sqlDialect.getSelectInsertId()); @@ -152,14 +148,18 @@ public abstract class DBAppenderBase<E> extends AppenderBase<E> { rs.close(); - if (idStatement != null) { - idStatement.close(); - idStatement = null; - } - + close(idStatement); + idStatement = null; + return eventId; } + void close(Statement statement) throws SQLException { + if(statement != null) { + statement.close(); + } + } + @Override public void stop() { super.stop(); diff --git a/logback-site/src/site/pages/manual/mdc.html b/logback-site/src/site/pages/manual/mdc.html index 9cf843b..b887d21 100644 --- a/logback-site/src/site/pages/manual/mdc.html +++ b/logback-site/src/site/pages/manual/mdc.html @@ -687,6 +687,14 @@ public class UserServletFilter implements Filter { </tr> <tr class="alt"> + <td><code>req.queryString</code></td> + <td> + as returned by <a + href="http://java.sun.com/j2ee/sdk_1.3/techdocs/api/javax/servlet/http/HttpServle…">getQueryString()</a> method + </td> + </tr> + + <tr> <td><code>req.userAgent</code></td> <td>value of the "User-Agent" header </td> diff --git a/pom.xml b/pom.xml index 3fcf402..d083acf 100755 --- a/pom.xml +++ b/pom.xml @@ -93,6 +93,12 @@ <artifactId>hsqldb</artifactId> <version>1.8.0.7</version> </dependency> + + <dependency> + <groupId>com.h2database</groupId> + <artifactId>h2</artifactId> + <version>1.2.131</version> + </dependency> <dependency> <groupId>org.apache.tomcat</groupId> <artifactId>catalina</artifactId> ----------------------------------------------------------------------- Summary of changes: .../ch/qos/logback/classic/ClassicConstants.java | 1 + .../classic/helpers/MDCInsertingServletFilter.java | 2 + .../ch/qos/logback/core/db/DBAppenderBase.java | 24 ++++++++++---------- logback-site/src/site/pages/manual/mdc.html | 8 ++++++ pom.xml | 6 +++++ 5 files changed, 29 insertions(+), 12 deletions(-) hooks/post-receive -- Logback: the generic, reliable, fast and flexible logging framework.

1 0

[JIRA] Created: (LBCORE-128) Please support implementation of binary log files in RollingFileAppender/FileAppender
by Joern Huxhorn (JIRA) 22 Mar '10

22 Mar '10

Please support implementation of binary log files in RollingFileAppender/FileAppender ------------------------------------------------------------------------------------- Key: LBCORE-128 URL: http://jira.qos.ch/browse/LBCORE-128 Project: logback-core Issue Type: Improvement Components: Appender Affects Versions: 0.9.17 Reporter: Joern Huxhorn Assignee: Logback dev list This was discussed briefly at http://marc.info/?l=logback-dev&m=124905434331308&w=2 and I forgot to file a ticket about this. Currently, RandomFileAppender => FileAppender => WriterAppender is using the following method in WriterAppender to actually write the data: protected void writerWrite(String s, boolean flush) throws IOException Please add an additional method like protected void writerWrite(byte[] bytes, boolean flush) throws IOException to write to the underlying stream directly. writerWrite(String, boolean) could call that method after performing the transformation internally, making this change transparent for the rest of the implementation. Using a binary format for logfiles could have tremendous performance impact as can be seen here: http://sourceforge.net/apps/trac/lilith/wiki/SerializationPerformance -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.qos.ch/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

6 42

[JIRA] Created: (LBCORE-145) option to create new file on start of SizeAndTimeBasedFNATP
by Ryan Cogswell (JIRA) 19 Mar '10

19 Mar '10

option to create new file on start of SizeAndTimeBasedFNATP ----------------------------------------------------------- Key: LBCORE-145 URL: http://jira.qos.ch/browse/LBCORE-145 Project: logback-core Issue Type: Improvement Components: Appender Affects Versions: 0.9.18 Reporter: Ryan Cogswell Assignee: Logback dev list Priority: Minor It would be nice to have the option of having the SizeAndTimeBasedFNATP always create a new file on a restart (only applicable when <File> is not specified -- only the <FileNamePattern>). Currently, if a process locks the latest file while the appender is stopped, then the FileAppender.openFile will fail if the file is still locked when the appender starts again. I'm currently accomplishing this by overriding SizeAndTimeBasedFNATP.start and using reflection to increment the currentPeriodsCounter. If a boolean option (e.g. "newFileOnStart") was added to SizeAndTimeBasedFNATP and computeCurrentPeriodsHighestCounterValue was modified to return whether or not a matching file was found, then the start method could be modified as shown below: if (tbrp.getParentsRawFileProperty() == null) { String regex = tbrp.fileNamePattern.toRegex(dateInCurrentPeriod); String stemRegex = FileFilterUtil.afterLastSlash(regex); // beginning of changes boolean fileExists = computeCurrentPeriodsHighestCounterValue(stemRegex); if (fileExists && newFileOnStart) { currentPeriodsCounter++; } // end of changes } -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.qos.ch/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

1 0

[JIRA] Created: (LBCLASSIC-196) ensure that layout setter is still accepted in FileAppender and co.
by Ceki Gulcu (JIRA) 19 Mar '10

19 Mar '10

ensure that layout setter is still accepted in FileAppender and co. ------------------------------------------------------------------- Key: LBCLASSIC-196 URL: http://jira.qos.ch/browse/LBCLASSIC-196 Project: logback-classic Issue Type: Task Components: appender Affects Versions: 0.9.19 Reporter: Ceki Gulcu Assignee: Logback dev list Priority: Critical Fix For: 0.9.19 -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.qos.ch/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

2 2

[GIT] Logback: the generic, reliable, fast and flexible logging framework. branch, master, updated. v0.9.18-103-g66fb732
by git-noreply＠pixie.qos.ch 19 Mar '10

19 Mar '10

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Logback: the generic, reliable, fast and flexible logging framework.". The branch, master has been updated via 66fb7325add00aea9f037fc9e813f07df4648518 (commit) from 0805c91cc85c6a36c38024e637f2aa2fe3b09a03 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://git.qos.ch/gitweb/?p=logback.git;a=commit;h=66fb7325add00aea9f037fc9… http://github.com/ceki/logback/commit/66fb7325add00aea9f037fc9e813f07df4648… commit 66fb7325add00aea9f037fc9e813f07df4648518 Author: Ceki Gulcu <ceki(a)qos.ch> Date: Fri Mar 19 15:34:32 2010 +0100 Fixes http://jira.qos.ch/browse/LBCLASSIC-196 diff --git a/logback-classic/src/test/input/joran/compatibility/layoutInsteadOfEncoder.xml b/logback-classic/src/test/input/joran/compatibility/layoutInsteadOfEncoder.xml index a50b1da..283ed0b 100644 --- a/logback-classic/src/test/input/joran/compatibility/layoutInsteadOfEncoder.xml +++ b/logback-classic/src/test/input/joran/compatibility/layoutInsteadOfEncoder.xml @@ -1,6 +1,6 @@ <configuration> - <appender name="FILE" class="ch.qos.logback.core.FileAppender"> + <appender name="LIOE" class="ch.qos.logback.core.FileAppender"> <file>target/test-output/layoutInsteadOfEncoder.log</file> <append>true</append> @@ -10,6 +10,6 @@ </appender> <root level="debug"> - <appender-ref ref="FILE" /> + <appender-ref ref="LIOE" /> </root> </configuration> \ No newline at end of file diff --git a/logback-classic/src/test/java/ch/qos/logback/classic/encoder/LayoutInsteadOfEncoderTest.java b/logback-classic/src/test/java/ch/qos/logback/classic/encoder/LayoutInsteadOfEncoderTest.java index f19db29..250c601 100644 --- a/logback-classic/src/test/java/ch/qos/logback/classic/encoder/LayoutInsteadOfEncoderTest.java +++ b/logback-classic/src/test/java/ch/qos/logback/classic/encoder/LayoutInsteadOfEncoderTest.java @@ -1,14 +1,18 @@ package ch.qos.logback.classic.encoder; import static ch.qos.logback.core.CoreConstants.CODES_URL; -import static org.junit.Assert.*; +import static org.junit.Assert.assertTrue; import org.junit.Before; import org.junit.Test; +import org.slf4j.Logger; import ch.qos.logback.classic.ClassicTestConstants; import ch.qos.logback.classic.LoggerContext; import ch.qos.logback.classic.joran.JoranConfigurator; +import ch.qos.logback.classic.spi.ILoggingEvent; +import ch.qos.logback.core.FileAppender; +import ch.qos.logback.core.encoder.LayoutWrappingEncoder; import ch.qos.logback.core.joran.spi.JoranException; import ch.qos.logback.core.status.Status; import ch.qos.logback.core.status.StatusChecker; @@ -34,7 +38,14 @@ public class LayoutInsteadOfEncoderTest { + "compatibility/layoutInsteadOfEncoder.xml"); StatusPrinter.print(loggerContext); StatusChecker checker = new StatusChecker(loggerContext); - assertTrue(checker.containsMatch(Status.ERROR, "This appender no longer admits a layout as a sub-component")); - assertTrue(checker.containsMatch(Status.ERROR, "See also "+CODES_URL+"#layoutInsteadOfEncoder for details")); + assertTrue(checker.containsMatch(Status.WARN, "This appender no longer admits a layout as a sub-component")); + assertTrue(checker.containsMatch(Status.WARN, "See also "+CODES_URL+"#layoutInsteadOfEncoder for details")); + + ch.qos.logback.classic.Logger root = (ch.qos.logback.classic.Logger) loggerContext.getLogger(Logger.ROOT_LOGGER_NAME); + FileAppender<ILoggingEvent> fileAppender = (FileAppender<ILoggingEvent>) root.getAppender("LIOE"); + assertTrue(fileAppender.isStarted()); + + LayoutWrappingEncoder<ILoggingEvent> lwe = (LayoutWrappingEncoder<ILoggingEvent>) fileAppender.getEncoder(); + } } diff --git a/logback-core/src/main/java/ch/qos/logback/core/OutputStreamAppender.java b/logback-core/src/main/java/ch/qos/logback/core/OutputStreamAppender.java index 373a917..0ba51b4 100644 --- a/logback-core/src/main/java/ch/qos/logback/core/OutputStreamAppender.java +++ b/logback-core/src/main/java/ch/qos/logback/core/OutputStreamAppender.java @@ -13,13 +13,15 @@ */ package ch.qos.logback.core; +import static ch.qos.logback.core.CoreConstants.CODES_URL; + import java.io.IOException; import java.io.OutputStream; import ch.qos.logback.core.encoder.Encoder; +import ch.qos.logback.core.encoder.LayoutWrappingEncoder; import ch.qos.logback.core.spi.DeferredProcessingAware; import ch.qos.logback.core.status.ErrorStatus; -import static ch.qos.logback.core.CoreConstants.CODES_URL; /** * OutputStreamAppender appends events to a {@link OutputStream}. This class @@ -87,9 +89,14 @@ public class OutputStreamAppender<E> extends UnsynchronizedAppenderBase<E> { } } - public void setLayout(Layout layout) { - addError("This appender no longer admits a layout as a sub-component, set an encoder instead."); - addError("See also "+CODES_URL+"#layoutInsteadOfEncoder for details"); + public void setLayout(Layout<E> layout) { + addWarn("This appender no longer admits a layout as a sub-component, set an encoder instead."); + addWarn("To ensure compatibility, wrapping your layout in LayoutWrappingEncoder."); + addWarn("See also "+CODES_URL+"#layoutInsteadOfEncoder for details"); + LayoutWrappingEncoder<E> lwe = new LayoutWrappingEncoder<E>(); + lwe.setLayout(layout); + lwe.setContext(context); + this.encoder = lwe; } @Override @@ -223,5 +230,5 @@ public class OutputStreamAppender<E> extends UnsynchronizedAppenderBase<E> { public void setEncoder(Encoder<E> encoder) { this.encoder = encoder; } - + } diff --git a/logback-core/src/test/java/ch/qos/logback/core/appender/ConsoleAppenderTest.java b/logback-core/src/test/java/ch/qos/logback/core/appender/ConsoleAppenderTest.java index a4e5063..19c8afa 100644 --- a/logback-core/src/test/java/ch/qos/logback/core/appender/ConsoleAppenderTest.java +++ b/logback-core/src/test/java/ch/qos/logback/core/appender/ConsoleAppenderTest.java @@ -32,7 +32,6 @@ import ch.qos.logback.core.encoder.NopEncoder; import ch.qos.logback.core.layout.DummyLayout; import ch.qos.logback.core.status.Status; import ch.qos.logback.core.status.StatusChecker; -import ch.qos.logback.core.util.StatusPrinter; public class ConsoleAppenderTest extends AbstractAppenderTest<Object> { diff --git a/logback-site/src/site/pages/codes.html b/logback-site/src/site/pages/codes.html index 3ab510d..6ca3d07 100644 --- a/logback-site/src/site/pages/codes.html +++ b/logback-site/src/site/pages/codes.html @@ -59,17 +59,17 @@ In practical terms, this means that configuration files need to be changed - from (BAD) + from (DEPRECATED) <pre class="prettyprint source"><appender name="FILE" class="ch.qos.logback.core.FileAppender"> <file>testFile.log</file> ... <layout class="ch.qos.logback.classic.PatternLayout"> - <pattern>%-4relative [%thread] %-5level %logger{35} - %msg%n</pattern> + <pattern>%msg%n</pattern> </layout> </appender> </pre> - or the shorter equivalent (BAD) + or the shorter equivalent (DEPRECATED) <pre class="prettyprint source"><appender name="FILE" class="ch.qos.logback.core.FileAppender"> <file>testFile.log</file> @@ -105,7 +105,11 @@ We hope to make up for this inconvenience with cool new features - which are only possible using encoders. + which are only possible using encoders. During a transition + period, layouts passed as parameter will be automatically wrapped by + an encoder so that configuration files in the old format (using a + layout instead of encoder) will continue to work unmodifed. + <hr/> diff --git a/logback-site/src/site/pages/news.html b/logback-site/src/site/pages/news.html index 644d52a..912666c 100644 --- a/logback-site/src/site/pages/news.html +++ b/logback-site/src/site/pages/news.html @@ -53,7 +53,10 @@ href="codes.html#layoutInsteadOfEncoder"> modifications to existing configuration files</a>. We hope to make up for this inconvenience with cool new features which are only possible using - encoders. + encoders. During a transition period, layouts passed as + parameter will be automatically wrapped by an encoder so that + configuration files in the old format (using a layout instead of + encoder) will continue to work unmodifed. </div> ----------------------------------------------------------------------- Summary of changes: .../joran/compatibility/layoutInsteadOfEncoder.xml | 4 ++-- .../encoder/LayoutInsteadOfEncoderTest.java | 17 ++++++++++++++--- .../ch/qos/logback/core/OutputStreamAppender.java | 17 ++++++++++++----- .../logback/core/appender/ConsoleAppenderTest.java | 1 - logback-site/src/site/pages/codes.html | 12 ++++++++---- logback-site/src/site/pages/news.html | 5 ++++- 6 files changed, 40 insertions(+), 16 deletions(-) hooks/post-receive -- Logback: the generic, reliable, fast and flexible logging framework.

1 0

[GIT] Logback: the generic, reliable, fast and flexible logging framework. branch, master, updated. v0.9.18-102-g0805c91
by git-noreply＠pixie.qos.ch 19 Mar '10

19 Mar '10

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Logback: the generic, reliable, fast and flexible logging framework.". The branch, master has been updated via 0805c91cc85c6a36c38024e637f2aa2fe3b09a03 (commit) via 13148e6ffc936dfafa2888c1bdb20f097494357e (commit) via 35a2ffe580a13ad82f6798edf396ef9d5138e65b (commit) via 2ff886bfdc0e731e8ed39e6da029c3c88d130b75 (commit) via ffeda045d43d98eeb6bb420e87df9e726b9d34f7 (commit) from 544292b758f9d296f581b6ec265fbeea954e6c6a (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://git.qos.ch/gitweb/?p=logback.git;a=commit;h=0805c91cc85c6a36c38024e6… http://github.com/ceki/logback/commit/0805c91cc85c6a36c38024e637f2aa2fe3b09… commit 0805c91cc85c6a36c38024e637f2aa2fe3b09a03 Merge: 544292b 13148e6 Author: Ceki Gulcu <ceki(a)qos.ch> Date: Fri Mar 19 13:48:09 2010 +0100 Merge branch 'dbname' ----------------------------------------------------------------------- Summary of changes: .gitignore | 4 +- .../java/ch/qos/logback/classic/db/DBAppender.java | 39 +++++++------ .../java/ch/qos/logback/classic/db/SQLBuilder.java | 60 ++++++++++++++++++++ .../qos/logback/classic/db/names/ColumnName.java | 24 ++++++++ .../classic/db/names/CustomDBNameResolver.java | 52 +++++++++++++++++ .../logback/classic/db/names/DBNameResolver.java | 20 +++++++ .../classic/db/names/DefaultDBNameResolver.java | 30 ++++++++++ .../ch/qos/logback/classic/db/names/TableName.java | 13 ++++ 8 files changed, 223 insertions(+), 19 deletions(-) create mode 100644 logback-classic/src/main/java/ch/qos/logback/classic/db/SQLBuilder.java create mode 100644 logback-classic/src/main/java/ch/qos/logback/classic/db/names/ColumnName.java create mode 100644 logback-classic/src/main/java/ch/qos/logback/classic/db/names/CustomDBNameResolver.java create mode 100644 logback-classic/src/main/java/ch/qos/logback/classic/db/names/DBNameResolver.java create mode 100644 logback-classic/src/main/java/ch/qos/logback/classic/db/names/DefaultDBNameResolver.java create mode 100644 logback-classic/src/main/java/ch/qos/logback/classic/db/names/TableName.java hooks/post-receive -- Logback: the generic, reliable, fast and flexible logging framework.

1 0