Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Excessive logging with akka.pattern.AskTimeoutException #46

Closed
KarelCemus opened this issue Apr 22, 2016 · 2 comments
Closed

Excessive logging with akka.pattern.AskTimeoutException #46

KarelCemus opened this issue Apr 22, 2016 · 2 comments
Assignees
Labels
Milestone

Comments

@KarelCemus
Copy link
Owner

When requests to redis timeouts due to, e.g., disconnection, there is unnecessary excessive logging. Such exceptions should be handled with more condensed log messages

10:40:54 [error] play.api.cache.redis SETEX command failed for key 'mykey'.
akka.pattern.AskTimeoutException: Ask timed out on [Actor[akka://application/user/$b#1419961090]] after [1000 ms]. Sender[null] sent message of type "brando.Request".
    at akka.pattern.PromiseActorRef$$anonfun$1.apply$mcV$sp(AskSupport.scala:604)
    at akka.actor.Scheduler$$anon$4.run(Scheduler.scala:122)
    at scala.concurrent.Future$InternalCallbackExecutor$.unbatchedExecute(Future.scala:601)
    at scala.concurrent.BatchingExecutor$class.execute(BatchingExecutor.scala:109)
    at scala.concurrent.Future$InternalCallbackExecutor$.execute(Future.scala:599)
    at akka.actor.LightArrayRevolverScheduler$TaskHolder.executeTask(LightArrayRevolverScheduler.scala:322)
    at akka.actor.LightArrayRevolverScheduler$$anon$4.executeBucket$1(LightArrayRevolverScheduler.scala:274)
    at akka.actor.LightArrayRevolverScheduler$$anon$4.nextTick(LightArrayRevolverScheduler.scala:278)
    at akka.actor.LightArrayRevolverScheduler$$anon$4.run(LightArrayRevolverScheduler.scala:230)
    at java.lang.Thread.run(Thread.java:745)

@KarelCemus KarelCemus self-assigned this Apr 22, 2016
@KarelCemus KarelCemus added this to the 1.2.1 milestone Apr 22, 2016
@KarelCemus
Copy link
Owner Author

Also related to #44, way to much logs when not connected

@KarelCemus
Copy link
Owner Author

Designed condensed log versions, see RecoveryPolicy for more details

Reduced logs in tests via policy customization

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant