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

jiaguo pushed a commit to branch HF-1415
in repository https://gitbox.apache.org/repos/asf/pinot.git

commit d0de7f5570b74cf0cb13eb6c9504b1bef37b4bb1
Author: Jia Guo <jia...@linkedin.com>
AuthorDate: Wed Sep 18 13:10:17 2024 -0700

    remove checks introduced in https://github.com/apache/pinot/issues/12945 
that is backward in compatible (#14016)
    
    (cherry picked from commit 2dd7ccabb72b33cc1355745eb6576045f4af8b17)
---
 .../java/org/apache/pinot/segment/spi/index/ForwardIndexConfig.java   | 4 ----
 1 file changed, 4 deletions(-)

diff --git 
a/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/ForwardIndexConfig.java
 
b/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/ForwardIndexConfig.java
index 757ca23688..1e713bec02 100644
--- 
a/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/ForwardIndexConfig.java
+++ 
b/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/ForwardIndexConfig.java
@@ -65,10 +65,6 @@ public class ForwardIndexConfig extends IndexConfig {
     _rawIndexWriterVersion = rawIndexWriterVersion == null ? 
DEFAULT_RAW_WRITER_VERSION : rawIndexWriterVersion;
     _compressionCodec = compressionCodec;
 
-    if (targetMaxChunkSize != null && !(_deriveNumDocsPerChunk || 
_rawIndexWriterVersion == 4)) {
-      throw new IllegalStateException(
-          "targetMaxChunkSize should only be used when deriveNumDocsPerChunk 
is true or rawIndexWriterVersion is 4");
-    }
     _targetMaxChunkSizeBytes = targetMaxChunkSize == null ? 
DEFAULT_TARGET_MAX_CHUNK_SIZE_BYTES
         : (int) DataSizeUtils.toBytes(targetMaxChunkSize);
     _targetMaxChunkSize =


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to