Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • O OS Core Common
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 22
    • Issues 22
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 4
    • Merge requests 4
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Open Subsurface Data Universe SoftwareOpen Subsurface Data Universe Software
  • Platform
  • System
  • Lib
  • core
  • OS Core Common
  • Merge requests
  • !85

Made the constructor of services public

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Muskan Srivastava requested to merge muskans-core into master May 18, 2021
  • Overview 3
  • Commits 2
  • Pipelines 4
  • Changes 7

This MR makes constructor of following classes as public: -

  • CrsConverterService
  • EntitlementsService
  • LegalService
  • SubscriptionService(notification)
  • PolicyService
  • SearchService
  • StorageService

Currently, there are 2 services - PartitionService and EntitlementsService V2, which have public constructors and rest have package-private. Making the constructor public will increase the accessibility and extensibility of Service classes ( used as placeholder). As of now, Factory classes which return an instance to Service (for example, EntitlementsFactory returns instance of EntitlementsService) acts as the only access point to Service classes. By making the constructor public of the above mentioned classes will enable us to extend those classes, write our own Factory classes that or use Http Client of our choice.
Impact :
This will enable us to: -

  1. Have parity in SDK.
  2. The factories can be overloaded. This will enable us to plug in HttpClient.
  3. Opens up Service classes being used as SDK to be able to be extensible and configurable.
Edited May 20, 2021 by Hema Vishnu Pola [Microsoft]
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: muskans-core