search

SpencerHaan / queuebacca

Apache License 2.0
5 stars 0 forks source link

Remove MessageSerializer from queuebacca-core #25

Open SpencerHaan opened 5 years ago

SpencerHaan commented 5 years ago

This interface doesn't really belong here, as it's dependent on the way the client is implemented. With SQS, JSON is a supported message body, and the SQS service uses JSON in it's API calls. This may not be the case with other brokers, for example one in memory, where there is no need for serialization at all. This should be consolidated into the JsonSerializer for the SQS module.

SpencerHaan commented 5 years ago

This is blocked by #10 as it's used to dump the message body for logging purposes.