-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Update: Reference messaging bridge pattern when refactoring to Azure Service Bus #4058
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
Conversation
|
@kbaley : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 53d1f3a: 💡 Validation status: suggestions
docs/industries/retail/migrate-ecommerce-solution.md
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. For any questions, please:
|
|
@MicrosoftDocs/cloud-architecture-content-team-pr-reviewers - Please review the proposed changes. If you want to move forward with the content, add a sign-off. #label:"pending-content-team/business-approval" |
|
Hi @kbaley - thanks very much for your contribution. Can you also work this implementation into your addition, to cite an example: https://github.com/Microsoft/Microsoft-Message-Bridge ? |
|
#label:"cact-review-in-progress" |
|
@BryanLa Done. I included another example implementation of it as well. |
Great, thank you @kbaley . We're planning to have a related discussion re: our patterns library in a couple of weeks, and possibly adding this as a new pattern article. We may move some of the implementation ideas to that article, and point to it from this one. Just wanted to give you a heads up that we'll probably hold off on this merge until we resolve that part. |
|
@BryanLa That makes sense. I thought the same thing and have started a Message Bridge article already. If you like, I can post that up as a starting point for you. I usually find it easier to edit an existing draft than to start from scratch. |
|
@BryanLa Actually, I see that a PR is already started for the new article by my colleague, Szymon. |
@kbaley - yes, exactly, and I was wondering if you two knew each other as well :) Sounds like a plan. We'll be reviewing with one of our devs in the next couple of weeks, but in the meantime, feel free to contribute to Szymon's branch. If our dev agrees we can move forward with this in our internal repo, then let you guys know when everything is set to publish. |
|
Anything new to report on this, @BryanLa. Can this PR be closed? |
|
@kbaley - thanks again for your contribution and patience. We discussed both this PR and PR 4068 today. The plan of action is to:
|
|
CACT #sign-off |
|
Learn Build status updates of commit abd57c3: ❌ Validation status: errorsPlease follow instructions here which may help to resolve issue.
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. For any questions, please:
|
For "lift and shift" migrations, when refactoring to Azure Service Bus, it can reduce the risk to migrate endpoints one at a time instead of all at once. This change makes reference to the "messaging bridge" pattern as a means of accomplishing that