Skip to content

Commit

Permalink
[FLINK-35359][config] Annotate the ConfigOptions that missing annotation
Browse files Browse the repository at this point in the history
  • Loading branch information
Sxnan authored and xintongsong committed May 23, 2024
1 parent 9ccfb65 commit 51da2d0
Show file tree
Hide file tree
Showing 14 changed files with 29 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@

package org.apache.flink.client.cli;

import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.configuration.ConfigOption;
import org.apache.flink.configuration.ConfigOptions;

Expand All @@ -27,6 +28,7 @@
import static org.apache.flink.configuration.ConfigOptions.key;

/** Artifact Fetch options. */
@PublicEvolving
public class ArtifactFetchOptions {

public static final ConfigOption<String> BASE_DIR =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,12 @@

package org.apache.flink.configuration;

import org.apache.flink.annotation.PublicEvolving;

import static org.apache.flink.configuration.ConfigOptions.key;

/** Configuration parameters for join/sort algorithms. */
@PublicEvolving
public class AlgorithmOptions {

public static final ConfigOption<Boolean> HASH_JOIN_BLOOM_FILTERS =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,13 @@
package org.apache.flink.configuration;

import org.apache.flink.annotation.Experimental;
import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.annotation.docs.Documentation;
import org.apache.flink.configuration.description.Description;
import org.apache.flink.configuration.description.TextElement;

/** A collection of all configuration options that relate to checkpoints and savepoints. */
@PublicEvolving
public class CheckpointingOptions {

// ------------------------------------------------------------------------
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@

package org.apache.flink.configuration;

import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.annotation.docs.Documentation;
import org.apache.flink.configuration.description.Description;
import org.apache.flink.configuration.description.TextElement;
Expand All @@ -30,6 +31,7 @@
import static org.apache.flink.configuration.description.TextElement.text;

/** The set of configuration options relating to high-availability settings. */
@PublicEvolving
public class HighAvailabilityOptions {

// ------------------------------------------------------------------------
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@

package org.apache.flink.configuration;

import org.apache.flink.annotation.Internal;
import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.annotation.docs.Documentation;
import org.apache.flink.configuration.description.Description;

Expand All @@ -29,7 +29,7 @@
import static org.apache.flink.configuration.description.TextElement.text;

/** Configuration parameters for REST communication. */
@Internal
@PublicEvolving
public class RestOptions {

private static final String REST_PORT_KEY = "rest.port";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
package org.apache.flink.configuration;

import org.apache.flink.annotation.Experimental;
import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.annotation.docs.Documentation;
import org.apache.flink.configuration.description.Description;
import org.apache.flink.core.security.token.DelegationTokenProvider;
Expand All @@ -34,6 +35,7 @@
import static org.apache.flink.util.Preconditions.checkNotNull;

/** The set of configuration options relating to security. */
@PublicEvolving
public class SecurityOptions {

public static final String DELEGATION_TOKEN_PROVIDER_PREFIX =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,13 +18,15 @@

package org.apache.flink.configuration;

import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.annotation.docs.Documentation;

import java.time.Duration;

import static org.apache.flink.configuration.ConfigOptions.key;

/** Configuration options to detect slow tasks. */
@PublicEvolving
public class SlowTaskDetectorOptions {

@Documentation.Section(Documentation.Sections.EXPERT_SCHEDULING)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,13 +18,15 @@

package org.apache.flink.cep.configuration;

import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.configuration.ConfigOption;
import org.apache.flink.configuration.ConfigOptions;
import org.apache.flink.util.TimeUtils;

import java.time.Duration;

/** CEP Cache Options. */
@PublicEvolving
public class CEPCacheOptions {

private CEPCacheOptions() {}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@

package org.apache.flink.metrics.influxdb;

import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.annotation.docs.Documentation;
import org.apache.flink.configuration.ConfigConstants;
import org.apache.flink.configuration.ConfigOption;
Expand All @@ -29,6 +30,7 @@
import org.influxdb.InfluxDB;

/** Config options for {@link InfluxdbReporter}. */
@PublicEvolving
@Documentation.SuffixOption(ConfigConstants.METRICS_REPORTER_PREFIX + "influxdb")
public class InfluxdbReporterOptions {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@

package org.apache.flink.metrics.prometheus;

import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.annotation.docs.Documentation;
import org.apache.flink.configuration.ConfigConstants;
import org.apache.flink.configuration.ConfigOption;
Expand All @@ -27,6 +28,7 @@
import org.apache.flink.configuration.description.TextElement;

/** Config options for the {@link PrometheusPushGatewayReporter}. */
@PublicEvolving
@Documentation.SuffixOption(ConfigConstants.METRICS_REPORTER_PREFIX + "prometheus")
public class PrometheusPushGatewayReporterOptions {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@

package org.apache.flink.runtime.highavailability;

import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.annotation.docs.Documentation;
import org.apache.flink.configuration.ConfigOption;
import org.apache.flink.configuration.ConfigOptions;
Expand All @@ -26,6 +27,7 @@
import org.apache.flink.configuration.description.TextElement;

/** The set of configuration options relating to the Job Result Store. */
@PublicEvolving
public class JobResultStoreOptions {
@Documentation.Section(Documentation.Sections.COMMON_HIGH_AVAILABILITY_JOB_RESULT_STORE)
public static final ConfigOption<String> STORAGE_PATH =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,12 @@

package org.apache.flink.runtime.shuffle;

import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.configuration.ConfigOption;
import org.apache.flink.configuration.ConfigOptions;

/** Options to configure shuffle service. */
@PublicEvolving
@SuppressWarnings("WeakerAccess")
public class ShuffleServiceOptions {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,13 +18,15 @@

package org.apache.flink.table.client.config;

import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.annotation.docs.Documentation;
import org.apache.flink.configuration.ConfigOption;
import org.apache.flink.configuration.ConfigOptions;
import org.apache.flink.table.api.config.TableConfigOptions;
import org.apache.flink.table.client.cli.parser.SyntaxHighlightStyle;

/** Options used in sql client. */
@PublicEvolving
public class SqlClientOptions {
private SqlClientOptions() {}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@

package org.apache.flink.yarn.configuration;

import org.apache.flink.annotation.PublicEvolving;
import org.apache.flink.configuration.ConfigOption;
import org.apache.flink.configuration.DescribedEnum;
import org.apache.flink.configuration.ExternalResourceOptions;
Expand All @@ -38,6 +39,7 @@
*
* <p>These options are not expected to be ever configured by users explicitly.
*/
@PublicEvolving
public class YarnConfigOptions {

/** The vcores used by YARN application master. */
Expand Down

0 comments on commit 51da2d0

Please sign in to comment.