-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Closed
Labels
ClientThis issue points to a problem in the data-plane of the library.This issue points to a problem in the data-plane of the library.Service Bus
Milestone
Description
- Kieran: [ServiceBus] Track2 Preview1 #9937 (comment)
- Class/Module improvement
- The question could be described as "which class should be in which module".
- Kieran: [ServiceBus] Track2 Preview1 #9937 (comment), [ServiceBus] Track2 Preview1 #9937 (comment)
- Naming of these _can_run, _is_live imply it should return bool while they don't
- need better naming or could consider a different implementation pattern
- Kieran: [ServiceBus] Track2 Preview1 #9937 (comment)
- concern about loop forever of the uamqp.client_ready()
- Anna: Message inheritance relationship:
- [ServiceBus] Track2 Preview1 #9937 (comment), [ServiceBus] Track2 Preview1 #9937 (comment)
- currently it's: ReceivedMessage => PeekMessage => Message
- do you want to keep outgoing/incoming message types separate?
- No for p1, duck typing, re-sendable received message
- Anna: [ServiceBus] Track2 Preview1 #9937 (comment)
- timeout for
sendneeded?
- timeout for
- Yijun: [ServiceBus] Track2 Preview1 #9937 (comment)
- security_token required for mgmt messages?
Metadata
Metadata
Labels
ClientThis issue points to a problem in the data-plane of the library.This issue points to a problem in the data-plane of the library.Service Bus