From b713f091b2948c253f55643eed56176a6fae5a7b Mon Sep 17 00:00:00 2001 From: Nik Everett Date: Thu, 29 Jun 2017 09:34:04 -0400 Subject: [PATCH] Fix warn that fails compile We don't throw any exception in `close` so we shouldn't declare that we do. Declaring that we do angers that java gods because we might throw `InterruptedException` which is evil to throw in `close`. Original commit: elastic/x-pack-elasticsearch@49ca8143d751d5e95c58055b98abd97a5739748b --- .../org/elasticsearch/xpack/sql/test/server/ProtoHandler.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sql-clients/test-utils/src/main/java/org/elasticsearch/xpack/sql/test/server/ProtoHandler.java b/sql-clients/test-utils/src/main/java/org/elasticsearch/xpack/sql/test/server/ProtoHandler.java index c4ac3c05ab1..9b520a2933e 100644 --- a/sql-clients/test-utils/src/main/java/org/elasticsearch/xpack/sql/test/server/ProtoHandler.java +++ b/sql-clients/test-utils/src/main/java/org/elasticsearch/xpack/sql/test/server/ProtoHandler.java @@ -83,7 +83,7 @@ public abstract class ProtoHandler implements HttpHandler, AutoCloseable { } @Override - public void close() throws Exception { + public void close() { // no-op } } \ No newline at end of file