sdk-java icon indicating copy to clipboard operation
sdk-java copied to clipboard

Contribution question for wrapper for RabbitmQ and Redis

Open amit2103 opened this issue 11 months ago • 8 comments

Hi,

We want to contribute a sdk wrapper for RabbitMQ and Redis pub sub. We think these two are very widely used and will be useful for a lot of developers. Just wanted to verify are there any constraints if we do so ? Can we go ahead and create the PR's

amit2103 avatar Mar 20 '24 08:03 amit2103

it will be a great help for many

ignitewala avatar Mar 20 '24 12:03 ignitewala

I believe for redis we will need a protocol binding in the specification first https://github.com/cloudevents/spec

For RabbitMQ, from my understanding it can support AMQP 1.0 with a plugin https://www.rabbitmq.com/docs/protocols and AMQP 1.0 is part of the spec https://github.com/cloudevents/spec/blob/main/cloudevents/bindings/amqp-protocol-binding.md, are you interesting in RabbitMQ using which protocol and protocol version ?

pierDipi avatar Mar 25 '24 12:03 pierDipi

cc @ikavgo

pierDipi avatar Mar 25 '24 12:03 pierDipi

thanks for pinging. RabbitMQ supports natively AMQP 0.9.1 and 1.0 as well as MQTT up to 5 and STOMP.

ikavgo avatar Mar 25 '24 14:03 ikavgo

Thanks , then perhaps we should start with the redis. So first step is to create a protocol binding :- https://github.com/cloudevents/spec and then code to actually implement it ?

amit2103 avatar Mar 25 '24 14:03 amit2103

thanks for pinging. RabbitMQ supports natively AMQP 0.9.1 and 1.0 as well as MQTT up to 5 and STOMP.

@amit2103 given this from Iliia, I guess AMQP with proton library could be already used for RabbitMQ https://github.com/cloudevents/sdk-java/tree/main/amqp or are you looking from something else/more?

pierDipi avatar Mar 25 '24 14:03 pierDipi

Thanks , then perhaps we should start with the redis. So first step is to create a protocol binding :- https://github.com/cloudevents/spec and then code to actually implement it ?

Yes, for the https://github.com/cloudevents/spec I would create an issue as well so that people can comment on it too

pierDipi avatar Mar 25 '24 14:03 pierDipi

Sounds good :) Thanks

amit2103 avatar Mar 25 '24 15:03 amit2103