Order caching after discovery client

ryanjbaxter · web-flow · commit 0e98e1b0d4f2 · 2026-01-13T16:44:59.000-05:00
Fixes #1630 Signed-off-by: Ryan Baxter <ryan.baxter@broadcom.com>
diff --git a/docs/modules/ROOT/pages/spring-cloud-commons/loadbalancer.adoc b/docs/modules/ROOT/pages/spring-cloud-commons/loadbalancer.adoc
@@ -118,8 +118,8 @@ public class CustomLoadBalancerConfiguration {
 			ConfigurableApplicationContext context) {
 		return ServiceInstanceListSupplier.builder()
 					.withDiscoveryClient()
-					.withWeighted()
 					.withCaching()
+					.withWeighted()
 					.build(context);
 	}
 }
@@ -139,8 +139,8 @@ public class CustomLoadBalancerConfiguration {
 			ConfigurableApplicationContext context) {
 		return ServiceInstanceListSupplier.builder()
 					.withDiscoveryClient()
-					.withWeighted(instance -> ThreadLocalRandom.current().nextInt(1, 101))
 					.withCaching()
+					.withWeighted(instance -> ThreadLocalRandom.current().nextInt(1, 101))
 					.build(context);
 	}
 }
@@ -403,8 +403,8 @@ public class CustomLoadBalancerConfiguration {
 			ConfigurableApplicationContext context) {
 		return ServiceInstanceListSupplier.builder()
 					.withDiscoveryClient()
-					.withSubset()
 					.withCaching()
+					.withSubset()
 					.build(context);
 	    }
 	}

Original file line number	Diff line number	Diff line change
`@@ -118,8 +118,8 @@ public class CustomLoadBalancerConfiguration {`
`118`	`118`	`ConfigurableApplicationContext context) {`
`119`	`119`	`return ServiceInstanceListSupplier.builder()`
`120`	`120`	`.withDiscoveryClient()`
`121`		`- .withWeighted()`
`122`	`121`	`.withCaching()`
	`122`	`+ .withWeighted()`
`123`	`123`	`.build(context);`
`124`	`124`	`}`
`125`	`125`	`}`
`@@ -139,8 +139,8 @@ public class CustomLoadBalancerConfiguration {`
`139`	`139`	`ConfigurableApplicationContext context) {`
`140`	`140`	`return ServiceInstanceListSupplier.builder()`
`141`	`141`	`.withDiscoveryClient()`
`142`		`- .withWeighted(instance -> ThreadLocalRandom.current().nextInt(1, 101))`
`143`	`142`	`.withCaching()`
	`143`	`+ .withWeighted(instance -> ThreadLocalRandom.current().nextInt(1, 101))`
`144`	`144`	`.build(context);`
`145`	`145`	`}`
`146`	`146`	`}`
`@@ -403,8 +403,8 @@ public class CustomLoadBalancerConfiguration {`
`403`	`403`	`ConfigurableApplicationContext context) {`
`404`	`404`	`return ServiceInstanceListSupplier.builder()`
`405`	`405`	`.withDiscoveryClient()`
`406`		`- .withSubset()`
`407`	`406`	`.withCaching()`
	`407`	`+ .withSubset()`
`408`	`408`	`.build(context);`
`409`	`409`	`}`
`410`	`410`	`}`