You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iceberg.apache.org by dw...@apache.org on 2022/08/05 19:10:51 UTC

[iceberg] branch master updated: API: Avoid unnecessary wrapping of CloseableIterable.iterator() (#5446)

This is an automated email from the ASF dual-hosted git repository.

dweeks pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iceberg.git


The following commit(s) were added to refs/heads/master by this push:
     new 3a82b4a39d API: Avoid unnecessary wrapping of CloseableIterable.iterator() (#5446)
3a82b4a39d is described below

commit 3a82b4a39d28259542125ee21c3549e541cefce7
Author: Eduard Tudenhöfner <et...@gmail.com>
AuthorDate: Fri Aug 5 21:10:47 2022 +0200

    API: Avoid unnecessary wrapping of CloseableIterable.iterator() (#5446)
---
 api/src/main/java/org/apache/iceberg/io/CloseableIterable.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/api/src/main/java/org/apache/iceberg/io/CloseableIterable.java b/api/src/main/java/org/apache/iceberg/io/CloseableIterable.java
index 7cea2c9d1e..294f66ad5a 100644
--- a/api/src/main/java/org/apache/iceberg/io/CloseableIterable.java
+++ b/api/src/main/java/org/apache/iceberg/io/CloseableIterable.java
@@ -99,7 +99,7 @@ public interface CloseableIterable<T> extends Iterable<T>, Closeable {
 
       @Override
       public CloseableIterator<E> iterator() {
-        return CloseableIterator.withClose(iterable.iterator());
+        return iterable.iterator();
       }
     };
   }