#19 NPE - NullPointerException ConnectionPool.notifyDataSourceIsDown(ConnectionPool.java:406)

rbygrave · rbygrave · commit 7ba64183e777 · 2019-02-03T21:50:27.000+13:00
Fix for issue introduced by #18
diff --git a/src/main/java/io/ebean/datasource/pool/ConnectionPool.java b/src/main/java/io/ebean/datasource/pool/ConnectionPool.java
@@ -403,7 +403,7 @@ private synchronized void notifyDataSourceIsDown(SQLException ex) {
     }
     if (!dataSourceDownAlertSent) {
       dataSourceDownAlertSent = true;
-      logger.error("FATAL: DataSourcePool [" + name + "] is down or has network error!!! message:" + ex.getMessage(), ex);
+      logger.error("FATAL: DataSourcePool [" + name + "] is down or has network error!!!", ex);
       if (notify != null) {
         notify.dataSourceDown(this, ex);
       }

Original file line number	Diff line number	Diff line change
`@@ -403,7 +403,7 @@ private synchronized void notifyDataSourceIsDown(SQLException ex) {`
`403`	`403`	`}`
`404`	`404`	`if (!dataSourceDownAlertSent) {`
`405`	`405`	`dataSourceDownAlertSent = true;`
`406`		`- logger.error("FATAL: DataSourcePool [" + name + "] is down or has network error!!! message:" + ex.getMessage(), ex);`
	`406`	`+ logger.error("FATAL: DataSourcePool [" + name + "] is down or has network error!!!", ex);`
`407`	`407`	`if (notify != null) {`
`408`	`408`	`notify.dataSourceDown(this, ex);`
`409`	`409`	`}`