#1252
  • Created on
    2011-10-03 10:28
  • Reported by
    Havoc Pennington
  • Status
    Invalid
  • Priority
    Normal (3)
  • Assigned to
    -
  • Milestone
    2.0-M1
  • Plan Level
    -
  • Due Date
    -

Attachments

Related Tickets

Add new subtask or bug

AMQP: Use implicits for AMQP serialization

Instead of this:

  def newRpcClient[O, I](connection: ActorRef,
                         exchangeName: String,
                         routingKey: String,
                         serializer: RpcClientSerializer[O, I]): RpcClient[O,I]


You could do something like:

 def newRpcClient[O, I](connection: ActorRef,
                         exchangeName: String,
                         routingKey: String)(implicit toBinary: ToBinary[O], fromBinary: FromBinary[I])


Then the serializer class can just go away, and it's easy for users to add support for serializing any type by just putting a ToBinary or FromBinary in scope.
User picture
  on 2011-10-03 10:45 *
By Jonas Bonér
Milestone set to 2.0
Updating tickets (#1244, #1246, #1249, #1250, #1251, #1252, #1255)
User picture
  on 2011-10-07 10:40 *
By viktorklang
Summary changed from Use implicits for AMQP serialization to AMQP: Use implicits for AMQP serialization
User picture
  on 2011-11-22 12:44 *
By viktorklang
Status changed from New to Invalid
AMQP should have a full overhaul.
User picture
  on 2011-12-14 20:49 *
By viktorklang
Milestone changed from 2.0 to 2.0-M1
Updating tickets (#1129, #1132, #1138, #1149, #1153, #1154, #1157, #1161, #1163, #1168, #1170, #1171, #1172, #1176, #1177, #1178, #1180, #1199, #1217, #1218, #1219, #1237, #1238, #1239, #1244, #1246, #1249, #1250, #1251, #1252, #1256, #1301, #1302, #1306, #1395, #1396, #1409, #1418, #455, #891, #895, #912, #956, #972, #1031, #1374, #880, #1125, #1146)