Skip to content

Implement refactoring improvements and adapt topology and custom endpoint caches and monitors #1371

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

Open
wants to merge 160 commits into
base: main
Choose a base branch
from

Conversation

aaron-congo
Copy link
Contributor

Description

Implements the following refactoring improvements:

  1. StorageService - a centralized, easy to access class that stores various types of information that our driver uses, eg custom endpoint info and topology info.
  2. MonitorService - a centralized, easy to access class that standardizes the submission and lifetime of the various monitors that our driver creates, eg custom endpoint monitors and topology monitors. Submitted monitors are stored under this service.
  3. ConnectionService - replaces the forceConnect functionality, providing the ability to create auxiliary ConnectionWrapper objects that no longer share a PluginService with the connection that is requesting the auxiliary connection.
  4. EventPublisher/EventSubscriber - a publish-subscribe pattern that allows the communication of generic events between interested driver components.

So far, the custom endpoint and topology caches and monitors have been adapted to use the StorageService and MonitorService. In the future, more caches and monitors will be adapted as well.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

aaron-congo and others added 30 commits May 20, 2025 13:33
…#1406)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
….1.21 (#1404)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…1409)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…21.0 (#1402)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…1408)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…viceContainer variables to servicesContainer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants