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

ClassCastException when migrating remote actors #6581

Closed
scabug opened this issue Oct 29, 2012 · 2 comments
Closed

ClassCastException when migrating remote actors #6581

scabug opened this issue Oct 29, 2012 · 2 comments
Assignees
Labels
Milestone

Comments

@scabug
Copy link

scabug commented Oct 29, 2012

Type casting in Actor.scala:64 fails with a ClassCastException when migrating remote actors. This prevents migration of remote actors at the early stage of actor migration.
The line that fails in Actor.scala:64:

def self: Actor = self(Scheduler).asInstanceOf[Actor]

The fix is very simple and has local effects. The invocations to Actor.self should be replaced with self(Scheduler) in the code of remote actors. The fix is already prepared and can be seen in:
http://github.com/vjovanov/scala/commit/626bd15b1335c03a85d7c3bd40cc324552108d8a

@scabug
Copy link
Author

scabug commented Oct 29, 2012

Imported From: https://issues.scala-lang.org/browse/SI-6581?orig=1
Reporter: @vjovanov
Affected Versions: 2.10.0-RC1
Other Milestones: 2.10.0

@scabug
Copy link
Author

scabug commented Oct 29, 2012

@adriaanm said:
to have this included, an argument needs to be made that

  • this is a regression
  • there is no workaround

the only reason we're willing to risk regressing due to a fix for a regression is if the latter cannot be worked around
otherwise, the risk of delaying the 2.10.0 release and eating into our 2.10.1 budget for fixing issues like this is too high

@scabug scabug closed this as completed Oct 31, 2012
@scabug scabug added the blocker label Apr 7, 2017
@scabug scabug added this to the 2.10.0-RC1 milestone Apr 7, 2017
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

2 participants