From 72c1fdacaccb1df47a10937b5953299b545eeb77 Mon Sep 17 00:00:00 2001
From: Thibault Duplessis <thibault.duplessis@gmail.com>
Date: Tue, 29 Apr 2014 14:27:51 +0200
Subject: [PATCH] reduce logging

---
 modules/ai/src/main/Client.scala          | 2 +-
 modules/analyse/src/main/Env.scala        | 1 -
 modules/game/src/main/CrosstableApi.scala | 4 +---
 modules/round/src/main/Env.scala          | 2 +-
 modules/round/src/main/Titivate.scala     | 4 ++--
 5 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/modules/ai/src/main/Client.scala b/modules/ai/src/main/Client.scala
index 053c6f3318..a7520cb61c 100644
--- a/modules/ai/src/main/Client.scala
+++ b/modules/ai/src/main/Client.scala
@@ -18,7 +18,7 @@ final class Client(
 
   private def withValidSituation[A](game: Game)(op: => Fu[A]): Fu[A] =
     if (game.toChess.situation playable true) op
-    else fufail("[ai stockfish] invalid game situation: " + game.toChess.situation)
+    else fufail("[ai stockfish] invalid position")
 
   def play(game: Game, level: Int): Fu[PlayResult] = withValidSituation(game) {
     for {
diff --git a/modules/analyse/src/main/Env.scala b/modules/analyse/src/main/Env.scala
index 338b3f337b..6712b9e397 100644
--- a/modules/analyse/src/main/Env.scala
+++ b/modules/analyse/src/main/Env.scala
@@ -41,7 +41,6 @@ final class Env(
     def receive = {
       case lila.hub.actorApi.ai.AutoAnalyse(gameId) =>
         val replyTo = sender
-        play.api.Logger("analyse").info(s"auto analyse http://lichess.org/$gameId")
         analyser.getOrGenerate(gameId, "lichess", admin = true, auto = true) onComplete {
           case Failure(err) =>
             play.api.Logger("analyse").warn(s"auto analyse http://lichess.org/$gameId failure: $err")
diff --git a/modules/game/src/main/CrosstableApi.scala b/modules/game/src/main/CrosstableApi.scala
index e6f5c0fdd3..d64b3f1ea4 100644
--- a/modules/game/src/main/CrosstableApi.scala
+++ b/modules/game/src/main/CrosstableApi.scala
@@ -20,9 +20,7 @@ final class CrosstableApi(coll: Coll) {
 
   def apply(u1: String, u2: String): Fu[Option[Crosstable]] =
     coll.find(select(u1, u2)).one[Crosstable] orElse create(u1, u2) recover {
-      case e: reactivemongo.core.commands.LastError =>
-        play.api.Logger("crosstable").info(e.getMessage)
-        none
+      case e: reactivemongo.core.commands.LastError if e.getMessage.contains("duplicate key error") => none
     }
 
   def add(game: Game): Funit = game.userIds.distinct.sorted match {
diff --git a/modules/round/src/main/Env.scala b/modules/round/src/main/Env.scala
index 9a6d5eff22..e6ea535914 100644
--- a/modules/round/src/main/Env.scala
+++ b/modules/round/src/main/Env.scala
@@ -147,7 +147,7 @@ final class Env(
   {
     import scala.concurrent.duration._
 
-    scheduler.future(0.33 hour, "game: finish by clock") {
+    scheduler.future(0.23 hour, "game: finish by clock") {
       titivate.finishByClock
     }
 
diff --git a/modules/round/src/main/Titivate.scala b/modules/round/src/main/Titivate.scala
index dd3ecdc228..99ae5cd61f 100644
--- a/modules/round/src/main/Titivate.scala
+++ b/modules/round/src/main/Titivate.scala
@@ -21,13 +21,13 @@ private[round] final class Titivate(
 
   def finishByClock: Funit =
     $primitive(Query.finishByClock, "_id", max = 5000.some)(_.asOpt[String]) addEffect { ids =>
-      logwarn("[titivate] Finish %d games by clock" format ids.size)
+      loginfo("[titivate] Finish %d games by clock" format ids.size)
       scheduler.throttle(100.millis)(ids) { id => roundMap ! Tell(id, Outoftime) }
     } void
 
   def finishAbandoned: Funit =
     $primitive(Query.abandoned, "_id", max = 5000.some)(_.asOpt[String]) addEffect { ids =>
-      logwarn("[titivate] Finish %d abandoned games" format ids.size)
+      loginfo("[titivate] Finish %d abandoned games" format ids.size)
       scheduler.throttle(100.millis)(ids) { id => roundMap ! Tell(id, Abandon) }
     } void
 }