Skip to content

Usage of Publisher Facade

Nikhil Singh[MicroSoft] requested to merge users/nikhil/UsageOfPublisherFacade into master

All Submissions:

  • [YES] I have added an explanation of what changes in this merge do and why we should include it?
  • [YES] Does the MR contain pipeline/ helm chart related changes?
  • [NA] I have updated the documentation accordingly.
  • [YES] I have added tests to cover my changes.
  • [NA] All new and existing tests passed.
  • [YES] My code follows the code style of this project.
  • [YES] I ran lint checks locally prior to submission.

What is the issue or story related to the change?

The MR contains changes to use publisher facade for publishing to message brokers. The schema service is a producer service for the notification service i.e it publishes notifications to Event grid that were getting pushed to notification service. Now the notification service has been migrated to Service bus and all the producer services are provided by a facade which they can use and start publishing to service bus.

Link to facade MR:

osdu/platform/system/lib/cloud/azure/os-core-lib-azure!143 (merged)

Link of related issue:

notification#25 (closed)

Does this introduce a breaking change?

  • [NO]

NA

Pending items

No

Edited by Nikhil Singh[MicroSoft]

Merge request reports