search

egetman / reactive-jms

Reactive JMS wrapper
Apache License 2.0
16 stars 0 forks source link

Support for CLIENT_ACKNOWLEDGE mode #2

Closed youngm closed 5 years ago

youngm commented 5 years ago

Thought much about supporting CLIENT_ACKNOWLEDGE mode? I see you have transaction support but not all queue providers that support JMS support transactions.

Thanks

egetman commented 5 years ago

Hi, @youngm. It sounds reasonable. There 2 simple ways of doing it: create a new constructor to UnicastJmsQueueSource, or modify the existing one (with transacted param). Do you wanna to create a PR?

youngm commented 5 years ago

@egetman I may. We were thinking of using this library for a project so I asked the question. But it looks like the project may go another direction. Feel free to close this issue and I'll reopen and submit a PR if we end up going this direction again. Thanks!

egetman commented 5 years ago

@youngm done.