Restore compatibility constructors for config APIs

Author: xiangfu0

pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/ForwardIndexConfig.java

@@ -75,7 +75,7 @@ public static ForwardIndexConfig getDefault() {
   }
 
   public static ForwardIndexConfig getDisabled() {
-    return new ForwardIndexConfig(true, null, null, null, null, null, null, null, null);
+    return new ForwardIndexConfig(true, (CompressionCodecSpec) null, null, null, null, null, null);
   }
 
   @JsonProperty("compressionCodec")
@@ -104,6 +104,20 @@ public ForwardIndexConfig(@Nullable Boolean disabled, @Nullable CompressionCodec
         rawIndexWriterVersion, targetMaxChunkSize, targetDocsPerChunk, configs);
   }
 
+  /**
+   * @deprecated Kept for binary compatibility with callers still constructing the config via legacy compression types.
+   */
+  @Deprecated
+  public ForwardIndexConfig(@Nullable Boolean disabled, @Nullable CompressionCodec compressionCodec,
+      @Nullable ChunkCompressionType chunkCompressionType, @Nullable DictIdCompressionType dictIdCompressionType,
+      @Nullable Boolean deriveNumDocsPerChunk, @Nullable Integer rawIndexWriterVersion,
+      @Nullable String targetMaxChunkSize, @Nullable Integer targetDocsPerChunk,
+      @Nullable Map<String, Object> configs) {
+    this(disabled, getActualCompressionCodecSpec(CompressionCodecSpec.fromCompressionCodec(compressionCodec),
+            chunkCompressionType, dictIdCompressionType), deriveNumDocsPerChunk, rawIndexWriterVersion,
+        targetMaxChunkSize, targetDocsPerChunk, configs);
+  }
+
   private ForwardIndexConfig(@Nullable Boolean disabled, @Nullable CompressionCodecSpec compressionCodecSpec,
       @Nullable Boolean deriveNumDocsPerChunk, @Nullable Integer rawIndexWriterVersion,
       @Nullable String targetMaxChunkSize, @Nullable Integer targetDocsPerChunk,

pinot-spi/src/main/java/org/apache/pinot/spi/config/table/FieldConfig.java

@@ -104,6 +104,19 @@ public FieldConfig(String name, EncodingType encodingType, @Nullable IndexType i
         timestampConfig, null, properties, null);
   }
 
+  /**
+   * Deprecated compatibility constructor retained for binary compatibility with callers that still construct
+   * {@link FieldConfig} directly with the enum-based compression codec.
+   */
+  @Deprecated
+  public FieldConfig(String name, EncodingType encodingType, @Nullable IndexType indexType,
+      @Nullable List<IndexType> indexTypes, @Nullable CompressionCodec compressionCodec,
+      @Nullable TimestampConfig timestampConfig, @Nullable JsonNode indexes,
+      @Nullable Map<String, String> properties, @Nullable JsonNode tierOverwrites) {
+    this(name, encodingType, indexType, indexTypes, CompressionCodecSpec.fromCompressionCodec(compressionCodec),
+        timestampConfig, indexes, properties, tierOverwrites);
+  }
+
   @JsonCreator
   public FieldConfig(@JsonProperty(value = "name", required = true) String name,
       @JsonProperty(value = "encodingType") EncodingType encodingType,