Github user ravipesala commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2173#discussion_r184365027
  
    --- Diff: 
store/search/src/main/java/org/apache/carbondata/store/worker/SearchRequestHandler.java
 ---
    @@ -105,8 +105,6 @@ public ShutdownResponse handleShutdown(ShutdownRequest 
request) {
           }
         } catch (InterruptedException e) {
           throw new IOException(e);
    -    } finally {
    -      reader.close();
    --- End diff --
    
    I think it is needed, otherwise memory and resources won't be cleaned


---

Reply via email to