Github user sujith71955 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2362#discussion_r193714706
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java ---
@@ -74,6 +74,8 @@ public boolean hasNext() throws IOException,
InterruptedException {
return false;
} else {
index++;
+ // current reader is closed
+ currentReader.close();
--- End diff --
Shall we close this in a finally block? else still there will be a chance
of memory leak.
---
