[docs-only] enhancement(docs): first draft of the activities service adr #9187
Conversation
|
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
fschade
changed the title
|
|
||
| ### Activity store | ||
|
|
||
| * Use a go-micro store to store the individual activities. |
There was a problem hiding this comment.
Can you stick to the ADR template and keep one headline per option?
Mabe make two ADRs one for store and one for format?
There was a problem hiding this comment.
can u explain what you mean, its a bit unclear to us?
There was a problem hiding this comment.
Or maybe even three ADRs: one for store, one for format, one for API.
| * Activities should be stored in a way that they can be queried efficiently. | ||
| * Activities should stay in place event if the resource is gone. | ||
| * Activities reflect the state at a given point in time and not the current state. | ||
| * The Service should only store a configurable number of activities per resource. |
There was a problem hiding this comment.
What's the story behind this statement? Usually I want to have all activity since the beginning of the universe.
There was a problem hiding this comment.
We were worried about amount of data store. But this limit can be deactivated if you have eternal space on your harddrive.
| ## Pros and Cons of the Options <!-- optional --> | ||
|
|
||
| * Activity store: | ||
| * (PRO) Introducing a new database type might be more efficient for storing activities. |
There was a problem hiding this comment.
Its a PRO for using another database
|
|
||
| ### Activity store | ||
|
|
||
| * Use a go-micro store to store the individual activities. |
There was a problem hiding this comment.
Or maybe even three ADRs: one for store, one for format, one for API.
feat(adr): Review suggestions Co-authored-by: Benedikt Kulmann <benedikt@kulmann.biz>
|
[docs-only] enhancement(docs): first draft of the activities service adr
Adds ADR for the new activities service, please add your thoughts below.
Fixes #9075