-
Notifications
You must be signed in to change notification settings - Fork 566
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SE Config can be created from MP Config #2060
SE Config can be created from MP Config #2060
Conversation
Looks like there are a few test failures and a SpotBugs issue to address. As for the implementation, I understand and agree with the overall approach, but can't really provide much feedback on the actual implementation. |
@@ -605,6 +584,7 @@ private EmptyConfigHolder() { | |||
.disableSystemPropertiesSource() | |||
.disableParserServices() | |||
.disableFilterServices() | |||
.changesExecutor(command -> {}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we could add some disableMutableSources
or something similar in the future
config/config-mp/src/test/java/io/helidon/config/mp/MpConfigTest.java
Outdated
Show resolved
Hide resolved
config/config-mp/src/test/java/io/helidon/config/mp/MpConfigTest.java
Outdated
Show resolved
Hide resolved
...rofile/messaging/src/test/java/io/helidon/microprofile/messaging/AdHocConfigBuilderTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
Empty config now does not create an executor service. Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
Added env var test. Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
eed6a3f
to
a5b2064
Compare
…ailure fix) Signed-off-by: Tomas Langer <tomas.langer@oracle.com>
Environment variables were not honored correctly - now we use MP config as the implementation and we have an SE config implementation fronting it when creating SE config from MP Config.
Resolves #2059