[ServiceBus] Track 2 Preview 1 Debt

[yunhaoling]

• 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 send needed?
• Yijun: [ServiceBus] Track2 Preview1 #9937 (comment)
  • security_token required for mgmt messages?