mojaloop / design-authority-project

This is the Issue and Decision Log for tracking mojaloop and related Designs
1 stars 2 forks source link

central-settlement merge into central-ledger #58

Closed ndonnan closed 3 years ago

ndonnan commented 4 years ago

Request:

Currently the central-settlement module is dependant on the central-ledger tables and is tightly integrated with central-ledger functionality. I would like to discuss the possible merging of the central-settlement repo into the central-ledger to aid in the development and maintenance of settlements development.

Artifacts:

Decision(s):

Follow-up:

Dependencies:

Accountability:

Notes:

claudio-viola commented 4 years ago

I second such a change, currently the database is centralised and at this stage we are unable to properly conduct PR review processes unless a change is committed and merged into central-ledger first hand. a quick example is a needed change in a central-settlement api which requires a new database attribute. in order for such a change to be completed a pull request would need to be made into central_ledger and merged and released before development and review CI/CD can continue in central-settlement service.

So we have two options, 1) merge back 2) invest into separating the database across microservices

godfreykutumela commented 3 years ago

Hi, @ndonnan @claudio-viola @elnyry-sam-k, can you please advise on the status of this issue? If it is not open for further discussion, I suggest moving it to closed or deferred.

ndonnan commented 3 years ago

This will be addressed as part of the event sourcing design

godfreykutumela commented 3 years ago

Thanks @ndonnan