-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider cloud events in relation to MQP payload and interop (W2AT feedback) #73
Comments
Raoult Beaudoin (at the W2AT meeting) mentioned possibility of interop or perhaps adoption of cloudevents. |
Original email: |
at first blush, after reading the spec. with some semi-farcical points given or taken away, more to illustrate what features are postive or negative and degree... admittedly highly subjective:
Overall score is about -4 so far... I think sticking with a pure MQP payload is probably simpler & better as losing server-side filtering is kind of crippling. We could always have an output adaptor for it... but perhaps others' might be more optimistic. |
I came to a similar conclusion as @petersilva . It seems to me that CloudEvents layer would add complexity without adding any special benefit. It does not really simplify adding support for other protocols. For AMQP it offers two different serialization methods (added complexity). |
further discussion here: https://github.com/wmo-im/wis2-notification-message |
https://github.com/cloudevents/spec
https://github.com/cloudevents/spec/blob/v1.0.2/cloudevents/bindings/mqtt-protocol-binding.md
The text was updated successfully, but these errors were encountered: