danny0405 commented on code in PR #12956:
URL: https://github.com/apache/hudi/pull/12956#discussion_r1990440710

##########
hudi-common/src/main/java/org/apache/hudi/common/util/collection/DiskMap.java:
##########
@@ -26,18 +26,22 @@
 import java.io.File;
 import java.io.IOException;
 import java.io.Serializable;
+import java.util.Collections;
+import java.util.HashSet;
 import java.util.Map;
+import java.util.Set;
 import java.util.UUID;
 import java.util.stream.Stream;
 
 /**
- * This interface provides the map interface for storing records in disk after 
they
+ * This interface provides the map interface for storing records in disk after
+ * they
  * spill over from memory. Used by {@link ExternalSpillableMap}.
  *
  * @param <T> The generic type of the keys
  * @param <R> The generic type of the values
  */
-public abstract class DiskMap<T extends Serializable, R> implements Map<T, R>, 
KeyFilteringIterable<T, R>, AutoCloseable {
+public abstract class DiskMap<T extends Serializable, R extends Serializable> 
implements Map<T, R>, Iterable<R> {

Review Comment:
   Why change this?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@hudi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to