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

tison pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/master by this push:
     new 2bab5adc1a1 [FLINK-29580][Connector/Pulsar] Remove 
pulsar.consumer.autoUpdatePartitionsIntervalSeconds option. (#21070)
2bab5adc1a1 is described below

commit 2bab5adc1a1d36d35ed971a007db9e311dc0363b
Author: Yufan Sheng <[email protected]>
AuthorDate: Tue Oct 18 13:27:38 2022 +0800

    [FLINK-29580][Connector/Pulsar] Remove 
pulsar.consumer.autoUpdatePartitionsIntervalSeconds option. (#21070)
---
 .../shortcodes/generated/pulsar_consumer_configuration.html        | 6 ------
 .../apache/flink/connector/pulsar/source/PulsarSourceOptions.java  | 7 -------
 .../connector/pulsar/source/config/PulsarSourceConfigUtils.java    | 5 -----
 3 files changed, 18 deletions(-)

diff --git 
a/docs/layouts/shortcodes/generated/pulsar_consumer_configuration.html 
b/docs/layouts/shortcodes/generated/pulsar_consumer_configuration.html
index e206ff4f20f..41f0b24b329 100644
--- a/docs/layouts/shortcodes/generated/pulsar_consumer_configuration.html
+++ b/docs/layouts/shortcodes/generated/pulsar_consumer_configuration.html
@@ -32,12 +32,6 @@
             <td>Boolean</td>
             <td>Buffering a large number of outstanding uncompleted chunked 
messages can bring memory pressure and it can be guarded by providing this 
<code class="highlighter-rouge">pulsar.consumer.maxPendingChunkedMessage</code> 
threshold. Once a consumer reaches this threshold, it drops the outstanding 
unchunked-messages by silently acknowledging if <code 
class="highlighter-rouge">pulsar.consumer.autoAckOldestChunkedMessageOnQueueFull</code>
 is true. Otherwise, it marks them for redel [...]
         </tr>
-        <tr>
-            
<td><h5>pulsar.consumer.autoUpdatePartitionsIntervalSeconds</h5></td>
-            <td style="word-wrap: break-word;">60</td>
-            <td>Integer</td>
-            <td>The interval (in seconds) of updating partitions. This only 
works if autoUpdatePartitions is enabled.</td>
-        </tr>
         <tr>
             <td><h5>pulsar.consumer.consumerName</h5></td>
             <td style="word-wrap: break-word;">(none)</td>
diff --git 
a/flink-connectors/flink-connector-pulsar/src/main/java/org/apache/flink/connector/pulsar/source/PulsarSourceOptions.java
 
b/flink-connectors/flink-connector-pulsar/src/main/java/org/apache/flink/connector/pulsar/source/PulsarSourceOptions.java
index 440e96b4f71..d0ef6dea11a 100644
--- 
a/flink-connectors/flink-connector-pulsar/src/main/java/org/apache/flink/connector/pulsar/source/PulsarSourceOptions.java
+++ 
b/flink-connectors/flink-connector-pulsar/src/main/java/org/apache/flink/connector/pulsar/source/PulsarSourceOptions.java
@@ -584,13 +584,6 @@ public final class PulsarSourceOptions {
                     .defaultValue(false)
                     .withDescription("If enabled, the consumer will 
automatically retry messages.");
 
-    public static final ConfigOption<Integer> 
PULSAR_AUTO_UPDATE_PARTITIONS_INTERVAL_SECONDS =
-            ConfigOptions.key(CONSUMER_CONFIG_PREFIX + 
"autoUpdatePartitionsIntervalSeconds")
-                    .intType()
-                    .defaultValue(60)
-                    .withDescription(
-                            "The interval (in seconds) of updating partitions. 
This only works if autoUpdatePartitions is enabled.");
-
     public static final ConfigOption<Boolean> 
PULSAR_REPLICATE_SUBSCRIPTION_STATE =
             ConfigOptions.key(CONSUMER_CONFIG_PREFIX + 
"replicateSubscriptionState")
                     .booleanType()
diff --git 
a/flink-connectors/flink-connector-pulsar/src/main/java/org/apache/flink/connector/pulsar/source/config/PulsarSourceConfigUtils.java
 
b/flink-connectors/flink-connector-pulsar/src/main/java/org/apache/flink/connector/pulsar/source/config/PulsarSourceConfigUtils.java
index 063d588decc..d76ba0a92cc 100644
--- 
a/flink-connectors/flink-connector-pulsar/src/main/java/org/apache/flink/connector/pulsar/source/config/PulsarSourceConfigUtils.java
+++ 
b/flink-connectors/flink-connector-pulsar/src/main/java/org/apache/flink/connector/pulsar/source/config/PulsarSourceConfigUtils.java
@@ -35,7 +35,6 @@ import java.util.concurrent.TimeUnit;
 
 import static java.util.concurrent.TimeUnit.MICROSECONDS;
 import static java.util.concurrent.TimeUnit.MILLISECONDS;
-import static java.util.concurrent.TimeUnit.SECONDS;
 import static 
org.apache.flink.connector.pulsar.common.config.PulsarOptions.PULSAR_ADMIN_URL;
 import static 
org.apache.flink.connector.pulsar.common.config.PulsarOptions.PULSAR_AUTH_PARAMS;
 import static 
org.apache.flink.connector.pulsar.common.config.PulsarOptions.PULSAR_AUTH_PARAM_MAP;
@@ -44,7 +43,6 @@ import static 
org.apache.flink.connector.pulsar.source.PulsarSourceOptions.PULSA
 import static 
org.apache.flink.connector.pulsar.source.PulsarSourceOptions.PULSAR_ACK_RECEIPT_ENABLED;
 import static 
org.apache.flink.connector.pulsar.source.PulsarSourceOptions.PULSAR_ACK_TIMEOUT_MILLIS;
 import static 
org.apache.flink.connector.pulsar.source.PulsarSourceOptions.PULSAR_AUTO_ACK_OLDEST_CHUNKED_MESSAGE_ON_QUEUE_FULL;
-import static 
org.apache.flink.connector.pulsar.source.PulsarSourceOptions.PULSAR_AUTO_UPDATE_PARTITIONS_INTERVAL_SECONDS;
 import static 
org.apache.flink.connector.pulsar.source.PulsarSourceOptions.PULSAR_CONSUMER_NAME;
 import static 
org.apache.flink.connector.pulsar.source.PulsarSourceOptions.PULSAR_CONSUMER_PROPERTIES;
 import static 
org.apache.flink.connector.pulsar.source.PulsarSourceOptions.PULSAR_CRYPTO_FAILURE_ACTION;
@@ -121,9 +119,6 @@ public final class PulsarSourceConfigUtils {
         configuration.useOption(PULSAR_READ_COMPACTED, builder::readCompacted);
         configuration.useOption(PULSAR_PRIORITY_LEVEL, builder::priorityLevel);
         
createDeadLetterPolicy(configuration).ifPresent(builder::deadLetterPolicy);
-        configuration.useOption(
-                PULSAR_AUTO_UPDATE_PARTITIONS_INTERVAL_SECONDS,
-                v -> builder.autoUpdatePartitionsInterval(v, SECONDS));
         configuration.useOption(PULSAR_RETRY_ENABLE, builder::enableRetry);
         configuration.useOption(
                 PULSAR_MAX_PENDING_CHUNKED_MESSAGE, 
builder::maxPendingChunkedMessage);

Reply via email to