This is an automated email from the ASF dual-hosted git repository.
xiangfu0 pushed a commit to branch release-1.6.0-rc
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/release-1.6.0-rc by this push:
new 04008879273 [1.6.0] Backport table-config deprecation metadata (#19978)
04008879273 is described below
commit 040088792734702f2beea2bfc1edc7d021ad6212
Author: Xiang Fu <[email protected]>
AuthorDate: Tue Oct 20 13:17:03 2026 -0700
[1.6.0] Backport table-config deprecation metadata (#19978)
Backport #18411. Record historical deprecation versions and removal intent
for 16 legacy table-config properties, with replacement Javadocs. Changes are
limited to annotations and documentation; runtime behavior and JSON
compatibility are unchanged.
(cherry picked from commit ec462d40c2783904ed4aadcf580fb07f3fe0a352)
---
.../java/org/apache/pinot/spi/config/table/DedupConfig.java | 6 ++++--
.../java/org/apache/pinot/spi/config/table/FieldConfig.java | 3 ++-
.../java/org/apache/pinot/spi/config/table/IndexingConfig.java | 6 +++++-
.../java/org/apache/pinot/spi/config/table/RoutingConfig.java | 2 ++
.../spi/config/table/SegmentsValidationAndRetentionConfig.java | 10 +++++-----
.../java/org/apache/pinot/spi/config/table/UpsertConfig.java | 9 ++++++---
.../table/assignment/InstanceReplicaGroupPartitionConfig.java | 2 ++
7 files changed, 26 insertions(+), 12 deletions(-)
diff --git
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/DedupConfig.java
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/DedupConfig.java
index 3c7bb6fe048..a3344d40661 100644
--- a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/DedupConfig.java
+++ b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/DedupConfig.java
@@ -164,7 +164,8 @@ public class DedupConfig extends BaseJsonConfig {
_metadataManagerConfigs = metadataManagerConfigs;
}
- @Deprecated
+ /// @deprecated Use `dedupConfig.preload` instead.
+ @Deprecated(since = "1.4.0", forRemoval = true)
public boolean isEnablePreload() {
return _enablePreload;
}
@@ -177,7 +178,8 @@ public class DedupConfig extends BaseJsonConfig {
}
}
- @Deprecated
+ /// @deprecated Use
`ingestionConfig.streamIngestionConfig.parallelSegmentConsumptionPolicy`
instead.
+ @Deprecated(since = "1.4.0", forRemoval = true)
public boolean isAllowDedupConsumptionDuringCommit() {
return _allowDedupConsumptionDuringCommit;
}
diff --git
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/FieldConfig.java
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/FieldConfig.java
index bbeec54847b..c8561570338 100644
--- a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/FieldConfig.java
+++ b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/FieldConfig.java
@@ -194,8 +194,9 @@ public class FieldConfig extends BaseJsonConfig {
return _encodingType;
}
+ /// @deprecated Use `fieldConfigList[].indexTypes` instead.
@Nullable
- @Deprecated
+ @Deprecated(since = "0.9.0", forRemoval = true)
public IndexType getIndexType() {
return !_indexTypes.isEmpty() ? _indexTypes.get(0) : null;
}
diff --git
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/IndexingConfig.java
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/IndexingConfig.java
index 8d8394b1833..b478fd05d42 100644
---
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/IndexingConfig.java
+++
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/IndexingConfig.java
@@ -133,6 +133,8 @@ public class IndexingConfig extends BaseJsonConfig {
_rangeIndexVersion = rangeIndexVersion;
}
+ /// @deprecated Use `tableIndexConfig.jsonIndexConfigs` instead.
+ @Deprecated(since = "0.12.0", forRemoval = true)
public List<String> getJsonIndexColumns() {
return _jsonIndexColumns;
}
@@ -157,7 +159,8 @@ public class IndexingConfig extends BaseJsonConfig {
_autoGeneratedInvertedIndex = autoGeneratedInvertedIndex;
}
- @Deprecated
+ /// @deprecated Remove this field; it is ignored.
+ @Deprecated(since = "1.6.0", forRemoval = true)
public boolean isCreateInvertedIndexDuringSegmentGeneration() {
return _createInvertedIndexDuringSegmentGeneration;
}
@@ -206,6 +209,7 @@ public class IndexingConfig extends BaseJsonConfig {
/// @deprecated Use `List<Map<String, String>> streamConfigs` from
///
[org.apache.pinot.spi.config.table.ingestion.IngestionConfig#getStreamIngestionConfig()]
@Nullable
+ @Deprecated(since = "0.7.1", forRemoval = true)
public Map<String, String> getStreamConfigs() {
return _streamConfigs;
}
diff --git
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/RoutingConfig.java
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/RoutingConfig.java
index b3cde3ea82b..54a63bb2038 100644
---
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/RoutingConfig.java
+++
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/RoutingConfig.java
@@ -54,7 +54,9 @@ public class RoutingConfig extends BaseJsonConfig {
_useFixedReplica = useFixedReplica;
}
+ /// @deprecated Use `routing.segmentPrunerTypes` and
`routing.instanceSelectorType` instead.
@Nullable
+ @Deprecated(since = "0.3.0", forRemoval = true)
public String getRoutingTableBuilderName() {
return _routingTableBuilderName;
}
diff --git
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/SegmentsValidationAndRetentionConfig.java
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/SegmentsValidationAndRetentionConfig.java
index 9027fd3ec57..73555e6d623 100644
---
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/SegmentsValidationAndRetentionConfig.java
+++
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/SegmentsValidationAndRetentionConfig.java
@@ -130,7 +130,7 @@ public class SegmentsValidationAndRetentionConfig extends
BaseJsonConfig {
/// @deprecated Use `segmentIngestionFrequency` from
///
[org.apache.pinot.spi.config.table.ingestion.IngestionConfig#getBatchIngestionConfig()]
- @Deprecated
+ @Deprecated(since = "0.8.0", forRemoval = true)
public String getSegmentPushFrequency() {
return _segmentPushFrequency;
}
@@ -142,7 +142,7 @@ public class SegmentsValidationAndRetentionConfig extends
BaseJsonConfig {
/// @deprecated Use `segmentIngestionType` from
///
[org.apache.pinot.spi.config.table.ingestion.IngestionConfig#getBatchIngestionConfig()]
- @Deprecated
+ @Deprecated(since = "0.8.0", forRemoval = true)
public String getSegmentPushType() {
return _segmentPushType;
}
@@ -165,7 +165,7 @@ public class SegmentsValidationAndRetentionConfig extends
BaseJsonConfig {
/// @deprecated Use \_replication instead
///
/// Will be deleted in future version of Pinot
- @Deprecated
+ @Deprecated(since = "1.1.0", forRemoval = true)
public String getReplicasPerPartition() {
return _replicasPerPartition;
}
@@ -179,7 +179,7 @@ public class SegmentsValidationAndRetentionConfig extends
BaseJsonConfig {
}
/// @deprecated Use
[org.apache.pinot.spi.config.table.assignment.InstanceAssignmentConfig] instead.
- @Deprecated
+ @Deprecated(since = "1.3.0", forRemoval = true)
public ReplicaGroupStrategyConfig getReplicaGroupStrategyConfig() {
return _replicaGroupStrategyConfig;
}
@@ -214,7 +214,7 @@ public class SegmentsValidationAndRetentionConfig extends
BaseJsonConfig {
}
/// @deprecated Use
[org.apache.pinot.spi.config.table.assignment.InstanceAssignmentConfig] instead
- @Deprecated
+ @Deprecated(since = "1.3.0", forRemoval = true)
public boolean isMinimizeDataMovement() {
return _minimizeDataMovement;
}
diff --git
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/UpsertConfig.java
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/UpsertConfig.java
index 0bd1149c3be..2dba6f92f4f 100644
---
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/UpsertConfig.java
+++
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/UpsertConfig.java
@@ -352,7 +352,8 @@ public class UpsertConfig extends BaseJsonConfig {
_metadataManagerConfigs = metadataManagerConfigs;
}
- @Deprecated
+ /// @deprecated Use `upsertConfig.snapshot` instead.
+ @Deprecated(since = "1.4.0", forRemoval = true)
public boolean isEnableSnapshot() {
return _enableSnapshot;
}
@@ -365,7 +366,8 @@ public class UpsertConfig extends BaseJsonConfig {
}
}
- @Deprecated
+ /// @deprecated Use `upsertConfig.preload` instead.
+ @Deprecated(since = "1.4.0", forRemoval = true)
public boolean isEnablePreload() {
return _enablePreload;
}
@@ -378,7 +380,8 @@ public class UpsertConfig extends BaseJsonConfig {
}
}
- @Deprecated
+ /// @deprecated Use
`ingestionConfig.streamIngestionConfig.parallelSegmentConsumptionPolicy`
instead.
+ @Deprecated(since = "1.4.0", forRemoval = true)
public boolean isAllowPartialUpsertConsumptionDuringCommit() {
return _allowPartialUpsertConsumptionDuringCommit;
}
diff --git
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/assignment/InstanceReplicaGroupPartitionConfig.java
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/assignment/InstanceReplicaGroupPartitionConfig.java
index 1bc40cba212..30628bdb68b 100644
---
a/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/assignment/InstanceReplicaGroupPartitionConfig.java
+++
b/pinot-spi/src/main/java/org/apache/pinot/spi/config/table/assignment/InstanceReplicaGroupPartitionConfig.java
@@ -102,6 +102,8 @@ public class InstanceReplicaGroupPartitionConfig extends
BaseJsonConfig {
return _numInstancesPerPartition;
}
+ /// @deprecated Remove this field; it will be removed in a future release.
+ @Deprecated(since = "1.1.0", forRemoval = true)
public boolean isMinimizeDataMovement() {
return _minimizeDataMovement;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
