Unnecesssary assert

DaveCTurner · DaveCTurner · commit a7a3e93611b0 · 2025-04-03T14:05:21.000+01:00
diff --git a/modules/repository-s3/src/main/java/org/elasticsearch/repositories/s3/S3Service.java b/modules/repository-s3/src/main/java/org/elasticsearch/repositories/s3/S3Service.java
@@ -252,8 +252,6 @@ protected S3ClientBuilder buildClientBuilder(S3ClientSettings clientSettings, Sd
 
     // TODO NOMERGE test this logic
     private Region getClientRegion(S3ClientSettings clientSettings) {
-        assert lifecycle.initialized() == false : lifecycle;
-
         if (Strings.hasLength(clientSettings.region)) {
             return Region.of(clientSettings.region);
         }

Original file line number	Diff line number	Diff line change
`@@ -252,8 +252,6 @@ protected S3ClientBuilder buildClientBuilder(S3ClientSettings clientSettings, Sd`
`252`	`252`
`253`	`253`	`// TODO NOMERGE test this logic`
`254`	`254`	`private Region getClientRegion(S3ClientSettings clientSettings) {`
`255`		`- assert lifecycle.initialized() == false : lifecycle;`
`256`		`-`
`257`	`255`	`if (Strings.hasLength(clientSettings.region)) {`
`258`	`256`	`return Region.of(clientSettings.region);`
`259`	`257`	`}`