refactor: remove duplicated close in columnarBatch (#100)

yingjianwu98 · stevie9868 · web-flow · commit 9c57adc118ef · 2025-10-30T20:51:16.000-07:00
Removing duplicate close for columnarBatch

Co-authored-by: stevie9868 &lt;yingjianwu2@email.com&gt;
diff --git a/lance-spark-base_2.12/src/main/java/com/lancedb/lance/spark/internal/LanceFragmentColumnarBatchScanner.java b/lance-spark-base_2.12/src/main/java/com/lancedb/lance/spark/internal/LanceFragmentColumnarBatchScanner.java
@@ -87,9 +87,6 @@ public ColumnarBatch getCurrentBatch() {
 
   @Override
   public void close() throws IOException {
-    if (currentColumnarBatch != null) {
-      currentColumnarBatch.close();
-    }
     if (currentColumnarBatch != null) {
       currentColumnarBatch.close();
     }

Original file line number	Diff line number	Diff line change
`@@ -87,9 +87,6 @@ public ColumnarBatch getCurrentBatch() {`
`87`	`87`
`88`	`88`	`@Override`
`89`	`89`	`public void close() throws IOException {`
`90`		`- if (currentColumnarBatch != null) {`
`91`		`- currentColumnarBatch.close();`
`92`		`- }`
`93`	`90`	`if (currentColumnarBatch != null) {`
`94`	`91`	`currentColumnarBatch.close();`
`95`	`92`	`}`