Skip to content

Commit 3676a53

Browse files
authored
fix: typo in AgregatePriorityListConfigProvider name (#3217)
1 parent c44c0a2 commit 3676a53

File tree

5 files changed

+17
-17
lines changed

5 files changed

+17
-17
lines changed

docs/content/en/docs/documentation/configuration.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -184,12 +184,12 @@ Operator operator = new Operator(ConfigLoader.getDefault().applyConfigs());
184184

185185
### Built-in Providers
186186

187-
| Provider | Source | Key mapping |
188-
|---|---|---|
189-
| `EnvVarConfigProvider` | `System.getenv()` | dots and hyphens → underscores, upper-cased (`josdk.check-crd``JOSDK_CHECK_CRD`) |
190-
| `PropertiesConfigProvider` | `java.util.Properties` or `.properties` file | key used as-is; use `PropertiesConfigProvider.systemProperties()` to read Java system properties |
191-
| `YamlConfigProvider` | YAML file | dot-separated key traverses nested mappings |
192-
| `AgregatePriorityListConfigProvider` | ordered list of providers | first non-empty result wins |
187+
| Provider | Source | Key mapping |
188+
|---------------------------------------|---|---|
189+
| `EnvVarConfigProvider` | `System.getenv()` | dots and hyphens → underscores, upper-cased (`josdk.check-crd``JOSDK_CHECK_CRD`) |
190+
| `PropertiesConfigProvider` | `java.util.Properties` or `.properties` file | key used as-is; use `PropertiesConfigProvider.systemProperties()` to read Java system properties |
191+
| `YamlConfigProvider` | YAML file | dot-separated key traverses nested mappings |
192+
| `AggregatePriorityListConfigProvider` | ordered list of providers | first non-empty result wins |
193193

194194
All string-based providers convert values to the target type automatically.
195195
Supported types: `String`, `Boolean`, `Integer`, `Long`, `Double`, `Duration` (ISO-8601, e.g. `PT30S`).
@@ -244,11 +244,11 @@ Only keys that are actually present in the source are applied; everything else r
244244
programmatic or annotation-based default.
245245

246246
You can also compose multiple sources with explicit priority using
247-
`AgregatePriorityListConfigProvider`:
247+
`AggregatePriorityListConfigProvider`:
248248

249249
```java
250250
var configLoader = new ConfigLoader(
251-
new AgregatePriorityListConfigProvider(List.of(
251+
new AggregatePriorityListConfigProvider(List.of(
252252
new EnvVarConfigProvider(), // highest priority
253253
PropertiesConfigProvider.systemProperties(),
254254
new YamlConfigProvider(Path.of("config/operator.yaml")) // lowest priority

operator-framework/src/main/java/io/javaoperatorsdk/operator/config/loader/ConfigLoader.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
import io.javaoperatorsdk.operator.api.config.ConfigurationServiceOverrider;
2828
import io.javaoperatorsdk.operator.api.config.ControllerConfigurationOverrider;
2929
import io.javaoperatorsdk.operator.api.config.LeaderElectionConfigurationBuilder;
30-
import io.javaoperatorsdk.operator.config.loader.provider.AgregatePriorityListConfigProvider;
30+
import io.javaoperatorsdk.operator.config.loader.provider.AggregatePriorityListConfigProvider;
3131
import io.javaoperatorsdk.operator.config.loader.provider.EnvVarConfigProvider;
3232
import io.javaoperatorsdk.operator.config.loader.provider.PropertiesConfigProvider;
3333
import io.javaoperatorsdk.operator.processing.event.rate.LinearRateLimiter;
@@ -166,7 +166,7 @@ public static ConfigLoader getDefault() {
166166

167167
public ConfigLoader() {
168168
this(
169-
new AgregatePriorityListConfigProvider(
169+
new AggregatePriorityListConfigProvider(
170170
List.of(new EnvVarConfigProvider(), PropertiesConfigProvider.systemProperties())),
171171
DEFAULT_CONTROLLER_KEY_PREFIX,
172172
DEFAULT_OPERATOR_KEY_PREFIX);

operator-framework/src/main/java/io/javaoperatorsdk/operator/config/loader/provider/AgregatePriorityListConfigProvider.java renamed to operator-framework/src/main/java/io/javaoperatorsdk/operator/config/loader/provider/AggregatePriorityListConfigProvider.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,11 @@
2424
* A {@link ConfigProvider} that delegates to an ordered list of providers. Providers are queried in
2525
* list order; the first non-empty result wins.
2626
*/
27-
public class AgregatePriorityListConfigProvider implements ConfigProvider {
27+
public class AggregatePriorityListConfigProvider implements ConfigProvider {
2828

2929
private final List<ConfigProvider> providers;
3030

31-
public AgregatePriorityListConfigProvider(List<ConfigProvider> providers) {
31+
public AggregatePriorityListConfigProvider(List<ConfigProvider> providers) {
3232
this.providers = List.copyOf(providers);
3333
}
3434

operator-framework/src/test/java/io/javaoperatorsdk/operator/config/loader/ConfigLoaderTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,7 @@ public <T> Optional<T> getValue(String key, Class<T> type) {
240240
// -- binding coverage -------------------------------------------------------
241241

242242
/**
243-
* Supported scalar types that AgregatePriorityListConfigProvider can parse from a string. Every
243+
* Supported scalar types that AggregatePriorityListConfigProvider can parse from a string. Every
244244
* binding's type must be one of these.
245245
*/
246246
private static final Set<Class<?>> SUPPORTED_TYPES =

operator-framework/src/test/java/io/javaoperatorsdk/operator/config/loader/provider/PriorityListConfigProviderTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,15 @@ private static PropertiesConfigProvider propsProvider(String key, String value)
3535
@Test
3636
void returnsEmptyWhenAllProvidersReturnEmpty() {
3737
var provider =
38-
new AgregatePriorityListConfigProvider(
38+
new AggregatePriorityListConfigProvider(
3939
List.of(new EnvVarConfigProvider(k -> null), propsProvider(null, null)));
4040
assertThat(provider.getValue("josdk.no.such.key", String.class)).isEmpty();
4141
}
4242

4343
@Test
4444
void firstProviderWins() {
4545
var provider =
46-
new AgregatePriorityListConfigProvider(
46+
new AggregatePriorityListConfigProvider(
4747
List.of(
4848
new EnvVarConfigProvider(k -> k.equals("JOSDK_TEST_KEY") ? "first" : null),
4949
propsProvider("josdk.test.key", "second")));
@@ -53,7 +53,7 @@ void firstProviderWins() {
5353
@Test
5454
void fallsBackToLaterProviderWhenEarlierReturnsEmpty() {
5555
var provider =
56-
new AgregatePriorityListConfigProvider(
56+
new AggregatePriorityListConfigProvider(
5757
List.of(
5858
new EnvVarConfigProvider(k -> null),
5959
propsProvider("josdk.test.key", "from-second")));
@@ -66,7 +66,7 @@ void respectsOrderWithThreeProviders() {
6666
var second = propsProvider("josdk.test.key", "from-second");
6767
var third = new EnvVarConfigProvider(k -> k.equals("JOSDK_TEST_KEY") ? "from-third" : null);
6868

69-
var provider = new AgregatePriorityListConfigProvider(List.of(first, second, third));
69+
var provider = new AggregatePriorityListConfigProvider(List.of(first, second, third));
7070
assertThat(provider.getValue("josdk.test.key", String.class)).hasValue("from-second");
7171
}
7272
}

0 commit comments

Comments
 (0)