Skip to content

AMQP 1.0 transport and AMQP type system mapping#157

Merged
duglin merged 6 commits into
cloudevents:masterfrom
clemensv:amqptransport
Jun 7, 2018
Merged

AMQP 1.0 transport and AMQP type system mapping#157
duglin merged 6 commits into
cloudevents:masterfrom
clemensv:amqptransport

Conversation

@clemensv
Copy link
Copy Markdown
Contributor

Proposed AMQP 1.0 transport binding and a type system mapping for the application properties section, analogous to the same for HTTP: #148

Signed-off-by: Clemens Vasters <clemensv@microsoft.com>
Clemens Vasters added 2 commits April 12, 2018 08:33
Signed-off-by: Clemens Vasters <clemensv@microsoft.com>
Signed-off-by: Clemens Vasters <clemensv@microsoft.com>
Comment thread amqp-transport-binding.md Outdated
## 3. AMQP Message Mapping

The content mode is chosen by the sender of the event, which is either the
requesting or the responding party. Gestures that might allow solicitation of
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gestures ... unclear - please reword

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Protocol interaction patterns that might
allow solicitation of events using a particular content mode might be defined
by an application, but are not defined here.

Clemens Vasters added 3 commits April 23, 2018 09:20
Signed-off-by: Clemens Vasters <clemensv@microsoft.com>
Signed-off-by: Clemens Vasters <clemensv@microsoft.com>
@yaronha
Copy link
Copy Markdown
Contributor

yaronha commented May 9, 2018

@clemensv do u have an example for an event (like u did for HTTP)?

also wasnt clear if we will be preserving the same notions like in HTTP (bin use AMQP Headers, structured using nested Json)?

Yaron

@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Jun 7, 2018

Approved on 6/7 call

@duglin duglin merged commit 8bc805b into cloudevents:master Jun 7, 2018
@longit644 longit644 mentioned this pull request Sep 1, 2018
@duglin duglin mentioned this pull request Nov 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants