Merge pull request #86 from njeuk/master

mauricio · mauricio · commit 888ca29c8f4b · 2014-04-06T23:54:45.000-03:00
Creating a unique named log per connection causes memory leak
diff --git a/postgresql-async/src/main/scala/com/github/mauricio/async/db/postgresql/PostgreSQLConnection.scala b/postgresql-async/src/main/scala/com/github/mauricio/async/db/postgresql/PostgreSQLConnection.scala
@@ -60,7 +60,7 @@ class PostgreSQLConnection
     executionContext
   )
   private final val currentCount = Counter.incrementAndGet()
-  private final val log = Log.getByName(s"${this.getClass.getName}:${currentCount}")
+  private final val log = Log.getByName(s"${this.getClass.getName}")
   private final val preparedStatementsCounter = new AtomicInteger()
   private final implicit val internalExecutionContext = executionContext
 

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ class PostgreSQLConnection`
`60`	`60`	`executionContext`
`61`	`61`	`)`
`62`	`62`	`private final val currentCount = Counter.incrementAndGet()`
`63`		`- private final val log = Log.getByName(s"${this.getClass.getName}:${currentCount}")`
	`63`	`+ private final val log = Log.getByName(s"${this.getClass.getName}")`
`64`	`64`	`private final val preparedStatementsCounter = new AtomicInteger()`
`65`	`65`	`private final implicit val internalExecutionContext = executionContext`
`66`	`66`