#1768
  • Created on
    2012-02-01 18:40
  • Reported by
    Patrik Nordwall
  • Status
    Fixed
  • Priority
    Normal (3)
  • Assigned to
    Patrik Nordwall
  • Milestone
    2.0-RC1
  • Plan Level
    -
  • Due Date
    -

Attachments

Related Tickets

Add new subtask or bug

ActorCell.actor null after fatal error in create

If the creation fails with fatal error the actor reference in actor cell is null while processing additional message. There is a check for this in invoke, with FIXME and warning log. It can be reproduced with attached sample.
User picture
  on 2012-02-01 18:40 *
By Patrik Nordwall
Attachment Errtest.scala added
file:ba4PICtqqr4yBBacwqjQXA
User picture
  on 2012-02-07 09:13 *
By viktorklang
What's the ticket about?
User picture
  on 2012-02-07 09:40 *
By Patrik Nordwall
If the creation (constructor) of an actor fails with fatal error (e.g. StackOverflowE) the actor reference in actor cell is null while processing additional message. It can be reproduced with attached sample.

There is a workaround now:
https://github.com/jboner/akka/blob/master/akka-actor/src/main/scala/akka/actor/ActorCell.scala#L480

but it is an indication that something else is broken.
User picture
  on 2012-02-07 10:08 *
By viktorklang
Then there's been a problem when adding NonFatal, as the actor should become suspended on failure:

https://github.com/jboner/akka/blob/master/akka-actor/src/main/scala/akka/actor/ActorCell.scala#L371
User picture
  on 2012-02-07 10:22 *
By Patrik Nordwall
Assigned to set to Patrik Nordwall
Status changed from New to Accepted
User picture
  on 2012-02-07 11:04 *
By Patrik Nordwall
(In revision:ac3868659305b2e1a891e7ac88ee66e2eb7ee9e8) System.exit on fatal error and removed actor null check. See #1799 and #1768

  • Config property jvmExitOnFatalError
  • System.exit in case of fatal error, such as OutOfMemoryError
  • Adjusted NonFatal extractor, ok with StackOverflowError, not ok with LinkageError
  • Removed the actor null check in ActorCell

Branch: wip-1799-jvmExitOnFatalError-patriknw
User picture
  on 2012-02-07 11:08 *
By Patrik Nordwall
Status changed from Accepted to Test
User picture
  on 2012-02-07 12:39 *
By Patrik Nordwall
(In revision:b021782eda16b924e1a1887741fc7be95c50aac4) Merge pull request #312 from jboner/wip-1799-jvmExitOnFatalError-patriknw

System.exit on fatal error and removed actor null check. See #1799 and #1768

Branch: master
User picture
  on 2012-02-07 12:40 *
By Patrik Nordwall
Status changed from Test to Fixed
User picture
  on 2012-02-10 10:40 *
By viktorklang
Milestone changed from 2.0 to 2.0-RC1
Updating tickets (#1625, #1768, #1777, #1789, #1794, #1795, #1796, #1799, #1803, #1805, #1809)