Merge pull request #83 from dangerousben/feature/logger-namespaces

Prepend class name to connection / state machine loggers.
finagle · Sep 18, 2018 · c6c1e75 · c6c1e75
2 parents fe28d32 + 229327e
commit c6c1e75
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/main/scala/com/twitter/finagle/postgres/connection/Connection.scala b/src/main/scala/com/twitter/finagle/postgres/connection/Connection.scala
@@ -11,7 +11,7 @@ import scala.collection.mutable.ListBuffer
  */
 class Connection(startState: State = AuthenticationRequired) {
   val id = Connection.nextId()
-  private[this] val logger = Logger(s"connection-$id")
+  private[this] val logger = Logger(s"${getClass.getName}.connection-$id")
   private[this] val stateMachine = new ConnectionStateMachine(startState, id)
 
 

diff --git a/src/main/scala/com/twitter/finagle/postgres/connection/ConnectionStateMachine.scala b/src/main/scala/com/twitter/finagle/postgres/connection/ConnectionStateMachine.scala
@@ -11,7 +11,7 @@ import scala.collection.mutable.ListBuffer
  * See associated Postgres documentation: http://www.postgresql.org/docs/9.0/static/protocol-flow.html
  */
 class ConnectionStateMachine(state: State = AuthenticationRequired, val id: Int) extends StateMachine[Message, PgResponse, State] {
-  private[this] val logger = Logger(s"psql state machine:$id")
+  private[this] val logger = Logger(s"${getClass.getName}.psql state machine:$id")
 
   startState(state)
 

diff --git a/src/main/scala/com/twitter/finagle/postgres/connection/StateMachine.scala b/src/main/scala/com/twitter/finagle/postgres/connection/StateMachine.scala
@@ -20,7 +20,7 @@ trait StateMachine[E, R, S] {
 
   val id: Int
 
-  private[this] val logger = Logger(s"state machine-$id")
+  private[this] val logger = Logger(s"${getClass.getName}.state machine-$id")
   private[this] var transitionFunction: Transition = Undefined
 
   @volatile private[this] var currentState: S = _