#2824
  • Created on
    2012-12-19 08:58
  • Reported by
    viktorklang
  • Status
    Fixed
  • Priority
    Normal (3)
  • Assigned to
    Patrik Nordwall
  • Milestone
    2.2-M2
  • Plan Level
    -
  • Due Date
    -

Attachments

Change logging of actor failures

remove the automatic logging of failures inside the failing actor, for instance: publish(Error(t, self.path.toString, clazz(actor), message)) in handleInvokeFailure.

Then add default logging of failures in the defaultStrategy, so if/when users want to customize log output they can just create their own supervisor strategy.
User picture
  on 2012-12-20 02:58 *
By rkuhn
Milestone set to Coltrane
User picture
  on 2013-01-22 06:18 *
By rkuhn
- RemoteLifecycleEvents vs. remoting actor failures
- skim clustering
User picture
  on 2013-01-22 06:20 *
By rkuhn
Estimate changed from Small to Medium
Sum of child estimates changed from 1.0 to 3.0
User picture
  on 2013-03-06 05:49 *
By viktorklang
Create a method inside Actor that returns the Directive, then the logging can be done from within there, then deprecate supervisorStrategy, default impl of the method should invoke supervisorStrategy
User picture
  on 2013-03-06 12:14 *
By Patrik Nordwall
Milestone changed from Coltrane to Current
User picture
  on 2013-03-06 12:14 *
By Patrik Nordwall
Assigned to set to Patrik Nordwall
Status changed from New to Accepted
User picture
  on 2013-03-07 07:31 *
By Patrik Nordwall
The comment about a new method inside Actor doesn't make sense to me, because SupervisorStrategy is much more and the directive. My proposal is here: https://github.com/akka/akka/pull/1228
User picture
  on 2013-03-08 07:22 *
By Patrik Nordwall
Status changed from Accepted to Test
User picture
  on 2013-03-11 09:38 *
By Patrik Nordwall
Status changed from Test to Fixed
User picture
  on 2013-03-18 08:07 *
By Patrik Nordwall
Milestone changed from Current to 2.2-M2