DRILL-8495: HTTP Caching Not Saving Pages (apache#2912)

Co-authored-by: Charles Givre <[email protected]>

contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/util/SimpleHttp.java

@@ -1013,7 +1013,7 @@ public static String getRequestAndStringResponse(String url) {
 
   /**
    *
-   * @param url
+   * @param url The input URL
    * @return an input stream which the caller is responsible for closing.
    */
   public static InputStream getRequestAndStreamResponse(String url) {
@@ -1029,9 +1029,9 @@ public static InputStream getRequestAndStreamResponse(String url) {
 
   /**
    *
-   * @param url
+   * @param url The input URL
    * @return response body which the caller is responsible for closing.
-   * @throws IOException
+   * @throws IOException If anything goes wrong, throws an IOException
    */
   public static ResponseBody makeSimpleGetRequest(String url) throws IOException {
     Request.Builder requestBuilder = new Request.Builder()
@@ -1047,14 +1047,9 @@ public static ResponseBody makeSimpleGetRequest(String url) throws IOException {
 
   @Override
   public void close() {
-    Cache cache;
     try {
-      cache = client.cache();
-      if (cache != null) {
-        cache.close();
-      }
       client.connectionPool().evictAll();
-    } catch (IOException e) {
+    } catch (Exception e) {
       logger.warn("Error closing cache. {}", e.getMessage());
     }
   }