Skip to content

Commit 9d5aa6f

Browse files
committed
change assert to logging
1 parent 4862340 commit 9d5aa6f

File tree

1 file changed

+14
-1
lines changed

1 file changed

+14
-1
lines changed

modules/repository-gcs/src/main/java/org/elasticsearch/repositories/gcs/GoogleCloudStorageHttpStatsCollector.java

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,18 @@
1313
import com.google.api.client.http.HttpResponseInterceptor;
1414

1515
import org.elasticsearch.common.blobstore.OperationPurpose;
16+
import org.elasticsearch.logging.LogManager;
17+
import org.elasticsearch.logging.Logger;
1618

1719
import java.util.regex.Pattern;
1820

1921
import static org.elasticsearch.repositories.gcs.GoogleCloudStorageOperationsStats.Operation;
2022
import static org.elasticsearch.repositories.gcs.GoogleCloudStorageOperationsStats.StatsTracker;
2123

2224
final class GoogleCloudStorageHttpStatsCollector implements HttpResponseInterceptor {
25+
26+
private static final Logger logger = LogManager.getLogger("GcpHttpStats");
27+
2328
private final StatsTracker stats;
2429
private final OperationPurpose purpose;
2530
private final Pattern getObjPattern;
@@ -78,6 +83,14 @@ public void interceptResponse(final HttpResponse response) {
7883
}
7984
default -> ignored = true;
8085
}
81-
assert ignored == false : "must handle response: " + request.getRequestMethod() + " " + path;
86+
if (ignored) {
87+
logger.debug(
88+
"not handling request:{} {} response:{} {}",
89+
request.getRequestMethod(),
90+
path,
91+
response.getStatusCode(),
92+
response.getStatusMessage()
93+
);
94+
}
8295
}
8396
}

0 commit comments

Comments
 (0)