From f159c0c49a7bad564f1fb6ba2758e4b89db4704a Mon Sep 17 00:00:00 2001
From: steven zhang <stevenz1985@outlook.com>
Date: Mon, 1 Feb 2021 20:51:57 +0800
Subject: [PATCH] [HUDI-1519] Improve minKey/maxKey computation in
 HoodieHFileWriter (#2427)

Co-authored-by: zhang wen <steven@stevendeMac-mini.local>
---
 .../org/apache/hudi/io/storage/HoodieHFileWriter.java | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/storage/HoodieHFileWriter.java b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/storage/HoodieHFileWriter.java
index 1358c35be..d77483caa 100644
--- a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/storage/HoodieHFileWriter.java
+++ b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/storage/HoodieHFileWriter.java
@@ -121,17 +121,10 @@ public class HoodieHFileWriter<T extends HoodieRecordPayload, R extends IndexedR
 
     if (hfileConfig.useBloomFilter()) {
       hfileConfig.getBloomFilter().add(recordKey);
-      if (minRecordKey != null) {
-        minRecordKey = minRecordKey.compareTo(recordKey) <= 0 ? minRecordKey : recordKey;
-      } else {
+      if (minRecordKey == null) {
         minRecordKey = recordKey;
       }
-
-      if (maxRecordKey != null) {
-        maxRecordKey = maxRecordKey.compareTo(recordKey) >= 0 ? maxRecordKey : recordKey;
-      } else {
-        maxRecordKey = recordKey;
-      }
+      maxRecordKey = recordKey;
     }
   }