This is an automated email from the ASF dual-hosted git repository.
xiangfu0 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new ec462d40c27 Mark deprecated table config properties with
@Deprecated(since, forRemoval = true) (#18411)
ec462d40c27 is described below
commit ec462d40c2783904ed4aadcf580fb07f3fe0a352
Author: Xiang Fu <[email protected]>
AuthorDate: Mon Oct 19 14:05:31 2026 -0700
Mark deprecated table config properties with @Deprecated(since, forRemoval
= true) (#18411)
Record on 16 deprecated table config getters the release that first
deprecated the property and that it is scheduled for removal, adding
@Deprecated where it was missing (tableIndexConfig.jsonIndexColumns,
tableIndexConfig.streamConfigs, routing.routingTableBuilderName and
replicaGroupPartitionConfig.minimizeDataMovement). Each getter also gets
a @deprecated Javadoc tag naming its replacement.
---
.../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 3e459eacf9f..83099b56517 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
@@ -220,8 +220,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 d88428b0a09..87a6dec046c 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
@@ -64,7 +64,9 @@ public class RoutingConfig extends BaseJsonConfig {
_partitionPruningPreparationThreshold =
partitionPruningPreparationThreshold;
}
+ /// @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 e503c2986f5..f8c59b64d59 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
@@ -185,7 +185,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;
}
@@ -197,7 +197,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;
}
@@ -220,7 +220,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;
}
@@ -234,7 +234,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;
}
@@ -269,7 +269,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]
