enqueue-dev icon indicating copy to clipboard operation
enqueue-dev copied to clipboard
verified publisher icon php-enqueue

Message serialization.

Open makasim opened this issue 6 years ago • 5 comments

needs https://github.com/queue-interop/queue-interop/pull/28 initial discussion https://github.com/queue-interop/queue-interop/issues/27 further discussion https://github.com/php-enqueue/enqueue-dev/issues/831

makasim avatar May 15 '19 07:05 makasim

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 24 '19 18:06 stale[bot]

@makasim What are we going to do with the PR to queue-interop? Want me to take over and create a new one?

Steveb-p avatar Sep 25 '19 07:09 Steveb-p

Honestly, I don't have a good solution. do not know how to properly address it

makasim avatar Sep 25 '19 11:09 makasim

Honestly, I don't have a good solution. do not know how to properly address it

As discussed in the PR in queue-interop, it has been proposed to drop type for body parameter in interface. This would open up ability for implementations to handle message serialization on their own :smile:

IMO interface is too tight and limits possible approaches. Some transports could actually benefit from this a lot - for example Redis transport can use IGB (don't recall the proper name on top of my head) serialization that is extension-based.

Steveb-p avatar Sep 25 '19 11:09 Steveb-p

and @makasim I'm only asking since @Sevavietl is currently not responding in queue-interop PR :smile:

Steveb-p avatar Sep 25 '19 11:09 Steveb-p