Making sure that akka-remote and akka-remote-tests pass

This commit is contained in:
Viktor Klang 2012-06-25 17:09:00 +02:00
parent 083039d379
commit 0f46157f19
7 changed files with 17 additions and 6 deletions

View file

@ -168,11 +168,8 @@ class Serialization(val system: ExtendedActorSystem) extends Extension {
* serializerMap is a Map whose keys is the class that is serializable and values is the serializer * serializerMap is a Map whose keys is the class that is serializable and values is the serializer
* to be used for that class. * to be used for that class.
*/ */
private val serializerMap: ConcurrentHashMap[Class[_], Serializer] = { private val serializerMap: ConcurrentHashMap[Class[_], Serializer] =
val serializerMap = new ConcurrentHashMap[Class[_], Serializer] (new ConcurrentHashMap[Class[_], Serializer] /: bindings) { case (map, (c, s)) map.put(c, s); map }
for ((c, s) bindings) serializerMap.put(c, s)
serializerMap
}
/** /**
* Maps from a Serializer Identity (Int) to a Serializer instance (optimization) * Maps from a Serializer Identity (Int) to a Serializer instance (optimization)

View file

@ -3,6 +3,8 @@
*/ */
package akka.remote.testconductor package akka.remote.testconductor
import language.postfixOps
import akka.actor.{ Actor, ActorRef, ActorSystem, LoggingFSM, Props } import akka.actor.{ Actor, ActorRef, ActorSystem, LoggingFSM, Props }
import RemoteConnection.getAddrString import RemoteConnection.getAddrString
import TestConductorProtocol._ import TestConductorProtocol._
@ -430,6 +432,7 @@ private[akka] class Controller(private var initialParticipants: Int, controllerP
case GetAddress(node) case GetAddress(node)
if (nodes contains node) sender ! ToClient(AddressReply(node, nodes(node).addr)) if (nodes contains node) sender ! ToClient(AddressReply(node, nodes(node).addr))
else addrInterest += node -> ((addrInterest get node getOrElse Set()) + sender) else addrInterest += node -> ((addrInterest get node getOrElse Set()) + sender)
case _: Done //FIXME what should happen?
} }
case op: CommandOp case op: CommandOp
op match { op match {

View file

@ -3,6 +3,8 @@
*/ */
package akka.remote.testconductor package akka.remote.testconductor
import language.implicitConversions
import org.jboss.netty.handler.codec.oneone.OneToOneEncoder import org.jboss.netty.handler.codec.oneone.OneToOneEncoder
import org.jboss.netty.channel.ChannelHandlerContext import org.jboss.netty.channel.ChannelHandlerContext
import org.jboss.netty.channel.Channel import org.jboss.netty.channel.Channel

View file

@ -3,6 +3,8 @@
*/ */
package akka.remote.testconductor package akka.remote.testconductor
import language.postfixOps
import java.net.InetSocketAddress import java.net.InetSocketAddress
import scala.annotation.tailrec import scala.annotation.tailrec

View file

@ -3,6 +3,8 @@
*/ */
package akka.remote.testconductor package akka.remote.testconductor
import language.postfixOps
import akka.actor.{ Actor, ActorRef, ActorSystem, LoggingFSM, Props } import akka.actor.{ Actor, ActorRef, ActorSystem, LoggingFSM, Props }
import RemoteConnection.getAddrString import RemoteConnection.getAddrString
import akka.util.duration._ import akka.util.duration._
@ -184,7 +186,7 @@ private[akka] class ClientFSM(name: RoleName, controllerAddr: InetSocketAddress)
case Event(Disconnected, _) case Event(Disconnected, _)
log.info("disconnected from TestConductor") log.info("disconnected from TestConductor")
throw new ConnectionFailure("disconnect") throw new ConnectionFailure("disconnect")
case Event(ToServer(Done), Data(Some(channel), _)) case Event(ToServer(_: Done), Data(Some(channel), _))
channel.write(Done) channel.write(Done)
stay stay
case Event(ToServer(msg), d @ Data(Some(channel), None)) case Event(ToServer(msg), d @ Data(Some(channel), None))
@ -232,6 +234,7 @@ private[akka] class ClientFSM(name: RoleName, controllerAddr: InetSocketAddress)
case TerminateMsg(exit) case TerminateMsg(exit)
System.exit(exit) System.exit(exit)
stay // needed because Java doesnt have Nothing stay // needed because Java doesnt have Nothing
case _: Done stay //FIXME what should happen?
} }
} }

View file

@ -3,6 +3,8 @@
*/ */
package akka.remote.testconductor package akka.remote.testconductor
import language.postfixOps
import akka.testkit.AkkaSpec import akka.testkit.AkkaSpec
import akka.actor.Props import akka.actor.Props
import akka.actor.AddressFromURIString import akka.actor.AddressFromURIString

View file

@ -3,6 +3,8 @@
*/ */
package akka.remote.testkit package akka.remote.testkit
import language.implicitConversions
import java.net.InetSocketAddress import java.net.InetSocketAddress
import com.typesafe.config.{ ConfigObject, ConfigFactory, Config } import com.typesafe.config.{ ConfigObject, ConfigFactory, Config }