Don't create a unique named logger per connection it causes a slow memory leak

njeuk · njeuk · commit 80876ee32937 · 2014-04-06T21:51:16.000+01:00
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`