diff options
| author | Jon Marius Venstad <jonmv@users.noreply.github.com> | 2024-05-27 11:14:53 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-05-27 11:14:53 +0200 |
| commit | 93b939f634279d9d270108649106270bdc6004c9 (patch) | |
| tree | 298a0176fa662f11541c47a7d813236b31b67a82 | |
| parent | 56796667a88c7976279a24803976b6744a208d62 (diff) | |
| parent | 25cfc78e4f85f50ab9f8a466c2c7a7a9b8873889 (diff) | |
Merge pull request #31309 from vespa-engine/jonmv/avoid-potential-NPE
Avoid potential NPE in feed result executor
| -rw-r--r-- | vespa-feed-client/src/main/java/ai/vespa/feed/client/impl/HttpRequestStrategy.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vespa-feed-client/src/main/java/ai/vespa/feed/client/impl/HttpRequestStrategy.java b/vespa-feed-client/src/main/java/ai/vespa/feed/client/impl/HttpRequestStrategy.java index 5fe59647038..63d061d85d3 100644 --- a/vespa-feed-client/src/main/java/ai/vespa/feed/client/impl/HttpRequestStrategy.java +++ b/vespa-feed-client/src/main/java/ai/vespa/feed/client/impl/HttpRequestStrategy.java @@ -154,7 +154,7 @@ class HttpRequestStrategy implements RequestStrategy { breaker.failure(thrown); if ( (thrown instanceof IOException) // General IO problems. // Thrown by HTTP2Session.StreamsState.reserveSlot, likely on GOAWAY from server - || (thrown instanceof IllegalStateException && thrown.getMessage().equals("session closed")) + || (thrown instanceof IllegalStateException && "session closed".equals(thrown.getMessage())) ) { log.log(FINER, thrown, () -> "Failed attempt " + attempt + " at " + request); return retry(request, attempt); |