You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Oct 14, 2020. It is now read-only.
Copy file name to clipboardExpand all lines: docs/adr/adr_0001.adoc
+5-5Lines changed: 5 additions & 5 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -107,9 +107,9 @@ The Execution Flow would then look something like this:
107
107
* Need to refactor the _persistence-elastic_ provider.
108
108
* The "`general implementation`" will be harder than the individual ones.
109
109
110
-
===== Solution Approach 1: Keep Split between Persistence Provider and MetaData Provider
110
+
===== Solution Approach 2: Keep Split between Persistence Provider and MetaData Provider
111
111
112
-
Keep PersistenceProvider as they are and introduce new "MetaDataProvider" CRD which gets executed before the PersistenceProviders by the operator.
112
+
Keep _PersistenceProvider_ as they are and introduce new _MetaDataProvider_ CRD which gets executed before the _PersistenceProviders_ by the __secureCodeBox operator_.
113
113
114
114
....
115
115
┌ Persistence Provider─ ─ ─ ─ ─ ─ ─ ─
@@ -124,12 +124,12 @@ Keep PersistenceProvider as they are and introduce new "MetaDataProvider" CRD wh
124
124
125
125
====== Pros
126
126
127
-
* Quicker to implement
128
-
* Might be worth it to have a separate concept for it
127
+
* Quicker to implement.
128
+
* Might be worth it to have a separate concept for it.
129
129
130
130
====== Cons
131
131
132
-
* Really worth introducing a new CRD for everything, especially when the are conceptually pretty close?
132
+
* Not sure if it worth to introduce a new CRD for everything, especially when it's conceptually pretty close to to something already existing.
133
133
134
134
==== Question 2: How Should the Execution Model Look like for Each?
0 commit comments