#717 Runtime activation of ETCD Config Sources #720
Conversation
| @@ -34,9 +35,10 @@ | |||
| */ | |||
| public class CachedEtcdConfigSource extends DefaultEtcdConfigSource { | |||
|
|
|||
| static boolean active = false; | |||
There was a problem hiding this comment.
A feladat leírásban, azzal hogy "active flag is needed with an extension of CachedEtcdConfigSource", arra utaltam, hogy ezt a runtime active flag kezelést egy leszármazottba kellene tenni, hogy aki ezt a CachedEtcdConfigSource-ot használja, azt ne befolyásolja. Így, hogy itt van default active = false-al, így rá van minden használó kényszerítve, hogy futásidőben is aktiválja a config source-t, de ez nem minden esetben kell, hogy így működjön, csak ha külön arra van igény.
There was a problem hiding this comment.
Elnézést, félre értettem a feladatot.
| @@ -34,9 +35,10 @@ | |||
| */ | |||
| public class CachedEtcdConfigSource extends DefaultEtcdConfigSource { | |||
There was a problem hiding this comment.
FilteredEtcdConfigSource is szerepel a feladatleírásban. Ahhoz is hasznos lenne egy ilyen megoldás a jövőre nézve. Vagy volt valami oka, hogy az nem lett kiterjesztve?
There was a problem hiding this comment.
Bocs, elkerülte a figyelmem, pótoltam.
There was a problem hiding this comment.
szerintem erre gondolt:
| [source,java] | ||
| ---- | ||
| CachedEtcdConfigSource.setActive(true); | ||
| ---- |
There was a problem hiding this comment.
majd release-notes-ot is írj kérlek
There was a problem hiding this comment.
Még nem találtam meg, hova kell. Tudsz segíteni?
No description provided.