Updated Architectural Decision Records (markdown)

Risto Lahtela 2019-10-22 10:43:44 +03:00
parent a88aed7b28
commit 48fd4163f5

@ -545,7 +545,7 @@ Impact: Increased database reliability and reduced exceptions from unpatched dat
</details> </details>
<details> <details>
<summary>📌 Removal of Proxy<->Server connection system</summary> <summary>📌 Removal of proxy<->server connection system</summary>
> **Problem:** Connection system causes large overhead on networks > **Problem:** Connection system causes large overhead on networks
> **Solution:** Remove the connection system > **Solution:** Remove the connection system
@ -555,3 +555,18 @@ Impact: Increased database reliability and reduced exceptions from unpatched dat
> Impact: Easier istallation > Impact: Easier istallation
</details> </details>
<details>
<summary>📌 Restructuring packages according to data flow</summary>
> **Problem:** Some classes are in packages that don't make sense
> **Solution:** Split packages based on data flow
>
> Here is a draft of the structure proposal
> https://github.com/plan-player-analytics/Plan/issues/1021#issuecomment-526286750
>
> Impact: Other plugins using internal classes in Plan break, might delay some people from updating. Data classes can be split by domain and gathering and delivery objects can be separated.
>
> Reality: Some classes from the old structure are still used in packages of the "wrong" domain, such as `Session` and `TaskSystem`
</details>