This is an automated email from the ASF dual-hosted git repository. lgallinat pushed a commit to branch feature/GEODE-4075 in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/feature/GEODE-4075 by this push: new 46abc78 GEODE-4075 Add @Experimental to all public classes and add experimental text to gfsh commands help 46abc78 is described below commit 46abc7876e47afe26c309578a002f769a9885d39 Author: Lynn Gallinat <lgalli...@pivotal.io> AuthorDate: Mon Dec 11 16:30:10 2017 -0800 GEODE-4075 Add @Experimental to all public classes and add experimental text to gfsh commands help --- .../geode/connectors/jdbc/internal/AbstractJdbcCallback.java | 2 ++ .../geode/connectors/jdbc/internal/ConnectionConfigBuilder.java | 3 +++ .../connectors/jdbc/internal/ConnectionConfigExistsException.java | 3 +++ .../jdbc/internal/ConnectionConfigNotFoundException.java | 3 +++ .../geode/connectors/jdbc/internal/ConnectionConfiguration.java | 3 +++ .../connectors/jdbc/internal/InternalJdbcConnectorService.java | 2 ++ .../geode/connectors/jdbc/internal/JdbcConnectorService.java | 2 ++ .../org/apache/geode/connectors/jdbc/internal/RegionMapping.java | 3 +++ .../geode/connectors/jdbc/internal/RegionMappingBuilder.java | 3 +++ .../connectors/jdbc/internal/RegionMappingExistsException.java | 3 +++ .../connectors/jdbc/internal/RegionMappingNotFoundException.java | 3 +++ .../java/org/apache/geode/connectors/jdbc/internal/SqlHandler.java | 2 ++ .../geode/connectors/jdbc/internal/cli/AlterConnectionCommand.java | 5 ++++- .../connectors/jdbc/internal/cli/AlterConnectionFunction.java | 2 ++ .../connectors/jdbc/internal/cli/AlterRegionMappingCommand.java | 6 +++++- .../connectors/jdbc/internal/cli/AlterRegionMappingFunction.java | 2 ++ .../connectors/jdbc/internal/cli/CreateConnectionCommand.java | 5 ++++- .../connectors/jdbc/internal/cli/CreateConnectionFunction.java | 2 ++ .../connectors/jdbc/internal/cli/CreateRegionMappingCommand.java | 6 ++++-- .../connectors/jdbc/internal/cli/CreateRegionMappingFunction.java | 2 ++ .../connectors/jdbc/internal/cli/DescribeConnectionCommand.java | 7 +++++-- .../connectors/jdbc/internal/cli/DescribeConnectionFunction.java | 2 ++ .../connectors/jdbc/internal/cli/DescribeRegionMappingCommand.java | 7 +++++-- .../jdbc/internal/cli/DescribeRegionMappingFunction.java | 2 ++ .../connectors/jdbc/internal/cli/DestroyConnectionCommand.java | 7 +++++-- .../connectors/jdbc/internal/cli/DestroyConnectionFunction.java | 2 ++ .../connectors/jdbc/internal/cli/DestroyRegionMappingCommand.java | 6 ++++-- .../connectors/jdbc/internal/cli/DestroyRegionMappingFunction.java | 2 ++ .../apache/geode/connectors/jdbc/internal/cli/JdbcCliFunction.java | 2 ++ .../geode/connectors/jdbc/internal/cli/ListConnectionCommand.java | 7 +++++-- .../geode/connectors/jdbc/internal/cli/ListConnectionFunction.java | 2 ++ .../connectors/jdbc/internal/cli/ListRegionMappingCommand.java | 6 ++++-- .../connectors/jdbc/internal/cli/ListRegionMappingFunction.java | 2 ++ 33 files changed, 99 insertions(+), 17 deletions(-) diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/AbstractJdbcCallback.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/AbstractJdbcCallback.java index 478edd0..64d370b 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/AbstractJdbcCallback.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/AbstractJdbcCallback.java @@ -16,9 +16,11 @@ package org.apache.geode.connectors.jdbc.internal; import java.util.Properties; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.CacheCallback; import org.apache.geode.internal.cache.InternalCache; +@Experimental public abstract class AbstractJdbcCallback implements CacheCallback { private volatile SqlHandler sqlHandler; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigBuilder.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigBuilder.java index 480517f..503b60d 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigBuilder.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigBuilder.java @@ -19,6 +19,9 @@ import static org.apache.geode.connectors.jdbc.internal.xml.JdbcConnectorService import java.util.HashMap; import java.util.Map; +import org.apache.geode.annotations.Experimental; + +@Experimental public class ConnectionConfigBuilder { private String name; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigExistsException.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigExistsException.java index 9094b10..de7b895 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigExistsException.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigExistsException.java @@ -14,6 +14,9 @@ */ package org.apache.geode.connectors.jdbc.internal; +import org.apache.geode.annotations.Experimental; + +@Experimental public class ConnectionConfigExistsException extends Exception { public ConnectionConfigExistsException() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigNotFoundException.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigNotFoundException.java index 0c74d3b..10830ea 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigNotFoundException.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfigNotFoundException.java @@ -14,6 +14,9 @@ */ package org.apache.geode.connectors.jdbc.internal; +import org.apache.geode.annotations.Experimental; + +@Experimental public class ConnectionConfigNotFoundException extends Exception { public ConnectionConfigNotFoundException() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfiguration.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfiguration.java index 51e2629..d993e54 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfiguration.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/ConnectionConfiguration.java @@ -19,6 +19,9 @@ import java.util.Collections; import java.util.Map; import java.util.Properties; +import org.apache.geode.annotations.Experimental; + +@Experimental public class ConnectionConfiguration implements Serializable { private static final Object USER = "user"; private static final Object PASSWORD = "password"; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/InternalJdbcConnectorService.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/InternalJdbcConnectorService.java index 43d1be5..223bf1f 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/InternalJdbcConnectorService.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/InternalJdbcConnectorService.java @@ -16,10 +16,12 @@ package org.apache.geode.connectors.jdbc.internal; import java.util.Set; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.Cache; import org.apache.geode.internal.cache.CacheService; import org.apache.geode.internal.cache.extension.Extension; +@Experimental public interface InternalJdbcConnectorService extends Extension<Cache>, CacheService { void createConnectionConfig(ConnectionConfiguration config) diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/JdbcConnectorService.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/JdbcConnectorService.java index 2761f51..7bf513b 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/JdbcConnectorService.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/JdbcConnectorService.java @@ -19,6 +19,7 @@ import java.util.Map; import java.util.Set; import java.util.concurrent.ConcurrentHashMap; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.Cache; import org.apache.geode.connectors.jdbc.internal.xml.JdbcConnectorServiceXmlGenerator; import org.apache.geode.internal.cache.CacheService; @@ -27,6 +28,7 @@ import org.apache.geode.internal.cache.extension.Extensible; import org.apache.geode.internal.cache.xmlcache.XmlGenerator; import org.apache.geode.management.internal.beans.CacheServiceMBeanBase; +@Experimental public class JdbcConnectorService implements InternalJdbcConnectorService { private final Map<String, ConnectionConfiguration> connectionsByName = new ConcurrentHashMap<>(); diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMapping.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMapping.java index dcc87a6..6b67a26 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMapping.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMapping.java @@ -18,6 +18,9 @@ import java.io.Serializable; import java.util.Collections; import java.util.Map; +import org.apache.geode.annotations.Experimental; + +@Experimental public class RegionMapping implements Serializable { private final String regionName; private final String pdxClassName; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingBuilder.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingBuilder.java index e4d0f76..300c2d4 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingBuilder.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingBuilder.java @@ -17,6 +17,9 @@ package org.apache.geode.connectors.jdbc.internal; import java.util.HashMap; import java.util.Map; +import org.apache.geode.annotations.Experimental; + +@Experimental public class RegionMappingBuilder { private static final String MAPPINGS_DELIMITER = ":"; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingExistsException.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingExistsException.java index 0696061..2b55721 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingExistsException.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingExistsException.java @@ -14,6 +14,9 @@ */ package org.apache.geode.connectors.jdbc.internal; +import org.apache.geode.annotations.Experimental; + +@Experimental public class RegionMappingExistsException extends Exception { public RegionMappingExistsException() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingNotFoundException.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingNotFoundException.java index cd88c71..7b0375e 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingNotFoundException.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/RegionMappingNotFoundException.java @@ -14,6 +14,9 @@ */ package org.apache.geode.connectors.jdbc.internal; +import org.apache.geode.annotations.Experimental; + +@Experimental public class RegionMappingNotFoundException extends Exception { public RegionMappingNotFoundException() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/SqlHandler.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/SqlHandler.java index b73698c..34703fc 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/SqlHandler.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/SqlHandler.java @@ -20,6 +20,7 @@ import java.sql.ResultSetMetaData; import java.sql.SQLException; import java.util.List; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.Operation; import org.apache.geode.cache.Region; import org.apache.geode.internal.cache.InternalCache; @@ -27,6 +28,7 @@ import org.apache.geode.pdx.PdxInstance; import org.apache.geode.pdx.PdxInstanceFactory; import org.apache.geode.pdx.internal.PdxInstanceImpl; +@Experimental public class SqlHandler { private ConnectionManager manager; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionCommand.java index bace38d..57def34 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionCommand.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionCommand.java @@ -20,6 +20,7 @@ import java.util.Set; import org.springframework.shell.core.annotation.CliCommand; import org.springframework.shell.core.annotation.CliOption; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.ResultCollector; import org.apache.geode.connectors.jdbc.internal.ConnectionConfigBuilder; import org.apache.geode.connectors.jdbc.internal.ConnectionConfiguration; @@ -35,10 +36,12 @@ import org.apache.geode.management.internal.configuration.domain.XmlEntity; import org.apache.geode.management.internal.security.ResourceOperation; import org.apache.geode.security.ResourcePermission; +@Experimental public class AlterConnectionCommand implements GfshCommand { + static final String EXPERIMENTAL = "(Experimental) "; static final String ALTER_JDBC_CONNECTION = "alter jdbc-connection"; static final String ALTER_JDBC_CONNECTION__HELP = - "Alter properties for an existing jdbc connection."; + EXPERIMENTAL + "Alter properties for an existing jdbc connection."; static final String ALTER_CONNECTION__NAME = "name"; static final String ALTER_CONNECTION__NAME__HELP = "Name of the connection to be altered."; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionFunction.java index c443dce..e2d6056 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterConnectionFunction.java @@ -16,6 +16,7 @@ package org.apache.geode.connectors.jdbc.internal.cli; import java.util.Map; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.ConnectionConfigNotFoundException; import org.apache.geode.connectors.jdbc.internal.ConnectionConfiguration; @@ -23,6 +24,7 @@ import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; import org.apache.geode.management.internal.cli.functions.CliFunctionResult; import org.apache.geode.management.internal.configuration.domain.XmlEntity; +@Experimental public class AlterConnectionFunction extends JdbcCliFunction<ConnectionConfiguration, CliFunctionResult> { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterRegionMappingCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterRegionMappingCommand.java index 112e9ca..2a9cc34 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterRegionMappingCommand.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterRegionMappingCommand.java @@ -20,6 +20,7 @@ import java.util.Set; import org.springframework.shell.core.annotation.CliCommand; import org.springframework.shell.core.annotation.CliOption; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.ResultCollector; import org.apache.geode.connectors.jdbc.internal.RegionMapping; import org.apache.geode.connectors.jdbc.internal.RegionMappingBuilder; @@ -35,9 +36,12 @@ import org.apache.geode.management.internal.configuration.domain.XmlEntity; import org.apache.geode.management.internal.security.ResourceOperation; import org.apache.geode.security.ResourcePermission; +@Experimental public class AlterRegionMappingCommand implements GfshCommand { + static final String EXPERIMENTAL = "(Experimental) "; static final String ALTER_MAPPING = "alter jdbc-mapping"; - static final String ALTER_MAPPING__HELP = "Alter properties for an existing jdbc mapping."; + static final String ALTER_MAPPING__HELP = + EXPERIMENTAL + "Alter properties for an existing jdbc mapping."; static final String ALTER_MAPPING__REGION_NAME = "region"; static final String ALTER_MAPPING__REGION_NAME__HELP = diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterRegionMappingFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterRegionMappingFunction.java index 773b5b8..f8ddb96 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterRegionMappingFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/AlterRegionMappingFunction.java @@ -16,6 +16,7 @@ package org.apache.geode.connectors.jdbc.internal.cli; import java.util.Map; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; import org.apache.geode.connectors.jdbc.internal.RegionMapping; @@ -23,6 +24,7 @@ import org.apache.geode.connectors.jdbc.internal.RegionMappingNotFoundException; import org.apache.geode.management.internal.cli.functions.CliFunctionResult; import org.apache.geode.management.internal.configuration.domain.XmlEntity; +@Experimental public class AlterRegionMappingFunction extends JdbcCliFunction<RegionMapping, CliFunctionResult> { AlterRegionMappingFunction() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionCommand.java index 6c03838..16eb261 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionCommand.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionCommand.java @@ -21,6 +21,7 @@ import org.apache.logging.log4j.Logger; import org.springframework.shell.core.annotation.CliCommand; import org.springframework.shell.core.annotation.CliOption; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.ResultCollector; import org.apache.geode.connectors.jdbc.internal.ConnectionConfigBuilder; import org.apache.geode.connectors.jdbc.internal.ConnectionConfiguration; @@ -37,12 +38,14 @@ import org.apache.geode.management.internal.configuration.domain.XmlEntity; import org.apache.geode.management.internal.security.ResourceOperation; import org.apache.geode.security.ResourcePermission; +@Experimental public class CreateConnectionCommand implements GfshCommand { private static final Logger logger = LogService.getLogger(); + static final String EXPERIMENTAL = "(Experimental) "; static final String CREATE_CONNECTION = "create jdbc-connection"; static final String CREATE_CONNECTION__HELP = - "Create a connection for communicating with a database through jdbc."; + EXPERIMENTAL + "Create a connection for communicating with a database through jdbc."; static final String CREATE_CONNECTION__NAME = "name"; static final String CREATE_CONNECTION__NAME__HELP = "Name of the connection to be created."; static final String CREATE_CONNECTION__URL = "url"; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionFunction.java index 7822d29..e50342b 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateConnectionFunction.java @@ -14,6 +14,7 @@ */ package org.apache.geode.connectors.jdbc.internal.cli; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.ConnectionConfigExistsException; import org.apache.geode.connectors.jdbc.internal.ConnectionConfiguration; @@ -21,6 +22,7 @@ import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; import org.apache.geode.management.internal.cli.functions.CliFunctionResult; import org.apache.geode.management.internal.configuration.domain.XmlEntity; +@Experimental public class CreateConnectionFunction extends JdbcCliFunction<ConnectionConfiguration, CliFunctionResult> { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateRegionMappingCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateRegionMappingCommand.java index cea4109..f3b725f 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateRegionMappingCommand.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateRegionMappingCommand.java @@ -20,6 +20,7 @@ import java.util.Set; import org.springframework.shell.core.annotation.CliCommand; import org.springframework.shell.core.annotation.CliOption; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.ResultCollector; import org.apache.geode.connectors.jdbc.internal.RegionMapping; import org.apache.geode.connectors.jdbc.internal.RegionMappingBuilder; @@ -35,11 +36,12 @@ import org.apache.geode.management.internal.configuration.domain.XmlEntity; import org.apache.geode.management.internal.security.ResourceOperation; import org.apache.geode.security.ResourcePermission; +@Experimental public class CreateRegionMappingCommand implements GfshCommand { - + static final String EXPERIMENTAL = "(Experimental) "; static final String CREATE_MAPPING = "create jdbc-mapping"; static final String CREATE_MAPPING__HELP = - "Create a mapping for a region for use with a JDBC database connection."; + EXPERIMENTAL + "Create a mapping for a region for use with a JDBC database connection."; static final String CREATE_MAPPING__REGION_NAME = "region"; static final String CREATE_MAPPING__REGION_NAME__HELP = "Name of the region the mapping is being created for."; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateRegionMappingFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateRegionMappingFunction.java index 14b0655..174b04d 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateRegionMappingFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateRegionMappingFunction.java @@ -14,6 +14,7 @@ */ package org.apache.geode.connectors.jdbc.internal.cli; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; import org.apache.geode.connectors.jdbc.internal.RegionMapping; @@ -21,6 +22,7 @@ import org.apache.geode.connectors.jdbc.internal.RegionMappingExistsException; import org.apache.geode.management.internal.cli.functions.CliFunctionResult; import org.apache.geode.management.internal.configuration.domain.XmlEntity; +@Experimental public class CreateRegionMappingFunction extends JdbcCliFunction<RegionMapping, CliFunctionResult> { CreateRegionMappingFunction() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeConnectionCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeConnectionCommand.java index 2d7cb65..8083be9 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeConnectionCommand.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeConnectionCommand.java @@ -25,6 +25,7 @@ import java.util.Set; import org.springframework.shell.core.annotation.CliCommand; import org.springframework.shell.core.annotation.CliOption; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.ResultCollector; import org.apache.geode.connectors.jdbc.internal.ConnectionConfiguration; import org.apache.geode.distributed.DistributedMember; @@ -37,10 +38,12 @@ import org.apache.geode.management.internal.cli.result.ResultBuilder; import org.apache.geode.management.internal.security.ResourceOperation; import org.apache.geode.security.ResourcePermission; +@Experimental public class DescribeConnectionCommand implements GfshCommand { - + static final String EXPERIMENTAL = "(Experimental) "; static final String DESCRIBE_CONNECTION = "describe jdbc-connection"; - static final String DESCRIBE_CONNECTION__HELP = "Describe the specified jdbc connection."; + static final String DESCRIBE_CONNECTION__HELP = + EXPERIMENTAL + "Describe the specified jdbc connection."; static final String DESCRIBE_CONNECTION__NAME = "name"; static final String DESCRIBE_CONNECTION__NAME__HELP = "Name of the jdbc connection to be described."; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeConnectionFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeConnectionFunction.java index 5a663a0..af9cd52 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeConnectionFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeConnectionFunction.java @@ -14,10 +14,12 @@ */ package org.apache.geode.connectors.jdbc.internal.cli; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.ConnectionConfiguration; import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; +@Experimental public class DescribeConnectionFunction extends JdbcCliFunction<String, ConnectionConfiguration> { DescribeConnectionFunction() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeRegionMappingCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeRegionMappingCommand.java index b442808..f70ed0d 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeRegionMappingCommand.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeRegionMappingCommand.java @@ -26,6 +26,7 @@ import java.util.Set; import org.springframework.shell.core.annotation.CliCommand; import org.springframework.shell.core.annotation.CliOption; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.ResultCollector; import org.apache.geode.connectors.jdbc.internal.RegionMapping; import org.apache.geode.distributed.DistributedMember; @@ -39,10 +40,12 @@ import org.apache.geode.management.internal.cli.result.TabularResultData; import org.apache.geode.management.internal.security.ResourceOperation; import org.apache.geode.security.ResourcePermission; +@Experimental public class DescribeRegionMappingCommand implements GfshCommand { - + static final String EXPERIMENTAL = "(Experimental) "; static final String DESCRIBE_MAPPING = "describe jdbc-mapping"; - static final String DESCRIBE_MAPPING__HELP = "Describe the specified jdbc mapping."; + static final String DESCRIBE_MAPPING__HELP = + EXPERIMENTAL + "Describe the specified jdbc mapping."; static final String DESCRIBE_MAPPING__REGION_NAME = "name"; static final String DESCRIBE_MAPPING__REGION_NAME__HELP = "Name of the jdbc mapping to be described."; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeRegionMappingFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeRegionMappingFunction.java index 2402671..56b410a 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeRegionMappingFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DescribeRegionMappingFunction.java @@ -14,10 +14,12 @@ */ package org.apache.geode.connectors.jdbc.internal.cli; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; import org.apache.geode.connectors.jdbc.internal.RegionMapping; +@Experimental public class DescribeRegionMappingFunction extends JdbcCliFunction<String, RegionMapping> { DescribeRegionMappingFunction() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionCommand.java index d06c31f..87b2e36 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionCommand.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionCommand.java @@ -20,6 +20,7 @@ import java.util.Set; import org.springframework.shell.core.annotation.CliCommand; import org.springframework.shell.core.annotation.CliOption; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.ResultCollector; import org.apache.geode.distributed.DistributedMember; import org.apache.geode.management.cli.CliMetaData; @@ -33,10 +34,12 @@ import org.apache.geode.management.internal.configuration.domain.XmlEntity; import org.apache.geode.management.internal.security.ResourceOperation; import org.apache.geode.security.ResourcePermission; +@Experimental public class DestroyConnectionCommand implements GfshCommand { - + static final String EXPERIMENTAL = "(Experimental) "; static final String DESTROY_CONNECTION = "destroy jdbc-connection"; - static final String DESTROY_CONNECTION__HELP = "Destroy/Remove the specified jdbc connection."; + static final String DESTROY_CONNECTION__HELP = + EXPERIMENTAL + "Destroy/Remove the specified jdbc connection."; static final String DESTROY_CONNECTION__NAME = "name"; static final String DESTROY_CONNECTION__NAME__HELP = "Name of the jdbc connection to be destroyed."; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionFunction.java index b29e78f..6a5e135 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyConnectionFunction.java @@ -14,12 +14,14 @@ */ package org.apache.geode.connectors.jdbc.internal.cli; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.ConnectionConfiguration; import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; import org.apache.geode.management.internal.cli.functions.CliFunctionResult; import org.apache.geode.management.internal.configuration.domain.XmlEntity; +@Experimental public class DestroyConnectionFunction extends JdbcCliFunction<String, CliFunctionResult> { DestroyConnectionFunction() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyRegionMappingCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyRegionMappingCommand.java index bb3439b..785a538 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyRegionMappingCommand.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyRegionMappingCommand.java @@ -20,6 +20,7 @@ import java.util.Set; import org.springframework.shell.core.annotation.CliCommand; import org.springframework.shell.core.annotation.CliOption; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.ResultCollector; import org.apache.geode.distributed.DistributedMember; import org.apache.geode.management.cli.CliMetaData; @@ -33,10 +34,11 @@ import org.apache.geode.management.internal.configuration.domain.XmlEntity; import org.apache.geode.management.internal.security.ResourceOperation; import org.apache.geode.security.ResourcePermission; +@Experimental public class DestroyRegionMappingCommand implements GfshCommand { - + static final String EXPERIMENTAL = "(Experimental) "; static final String DESTROY_MAPPING = "destroy jdbc-mapping"; - static final String DESTROY_MAPPING__HELP = "Destroy the specified mapping."; + static final String DESTROY_MAPPING__HELP = EXPERIMENTAL + "Destroy the specified mapping."; static final String DESTROY_MAPPING__REGION_NAME = "region"; static final String DESTROY_MAPPING__REGION_NAME__HELP = "Name of the region mapping to destroy."; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyRegionMappingFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyRegionMappingFunction.java index 79726d6..0b0703f 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyRegionMappingFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/DestroyRegionMappingFunction.java @@ -14,12 +14,14 @@ */ package org.apache.geode.connectors.jdbc.internal.cli; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; import org.apache.geode.connectors.jdbc.internal.RegionMapping; import org.apache.geode.management.internal.cli.functions.CliFunctionResult; import org.apache.geode.management.internal.configuration.domain.XmlEntity; +@Experimental public class DestroyRegionMappingFunction extends JdbcCliFunction<String, CliFunctionResult> { DestroyRegionMappingFunction() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/JdbcCliFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/JdbcCliFunction.java index 23418e9..ab667a3 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/JdbcCliFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/JdbcCliFunction.java @@ -14,12 +14,14 @@ */ package org.apache.geode.connectors.jdbc.internal.cli; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.Function; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; import org.apache.geode.internal.InternalEntity; import org.apache.geode.management.internal.configuration.domain.XmlEntity; +@Experimental public abstract class JdbcCliFunction<T1, T2> implements Function<T1>, InternalEntity { private final FunctionContextArgumentProvider argumentProvider; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListConnectionCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListConnectionCommand.java index 8da6e93..52fb133 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListConnectionCommand.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListConnectionCommand.java @@ -21,6 +21,7 @@ import java.util.Set; import org.springframework.shell.core.annotation.CliCommand; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.ResultCollector; import org.apache.geode.connectors.jdbc.internal.ConnectionConfiguration; import org.apache.geode.distributed.DistributedMember; @@ -33,10 +34,12 @@ import org.apache.geode.management.internal.cli.result.TabularResultData; import org.apache.geode.management.internal.security.ResourceOperation; import org.apache.geode.security.ResourcePermission; +@Experimental public class ListConnectionCommand implements GfshCommand { - + static final String EXPERIMENTAL = "(Experimental) "; static final String LIST_JDBC_CONNECTION = "list jdbc-connections"; - static final String LIST_JDBC_CONNECTION__HELP = "Display jdbc connections for all members."; + static final String LIST_JDBC_CONNECTION__HELP = + EXPERIMENTAL + "Display jdbc connections for all members."; static final String LIST_OF_CONNECTIONS = "List of connections"; static final String NO_CONNECTIONS_FOUND = "No connections found"; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListConnectionFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListConnectionFunction.java index 54f058e..20fdb53 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListConnectionFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListConnectionFunction.java @@ -16,10 +16,12 @@ package org.apache.geode.connectors.jdbc.internal.cli; import java.util.Set; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.ConnectionConfiguration; import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; +@Experimental public class ListConnectionFunction extends JdbcCliFunction<Void, ConnectionConfiguration[]> { ListConnectionFunction() { diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListRegionMappingCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListRegionMappingCommand.java index 67df211..c2c34ad 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListRegionMappingCommand.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListRegionMappingCommand.java @@ -21,6 +21,7 @@ import java.util.Set; import org.springframework.shell.core.annotation.CliCommand; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.ResultCollector; import org.apache.geode.connectors.jdbc.internal.RegionMapping; import org.apache.geode.distributed.DistributedMember; @@ -33,10 +34,11 @@ import org.apache.geode.management.internal.cli.result.TabularResultData; import org.apache.geode.management.internal.security.ResourceOperation; import org.apache.geode.security.ResourcePermission; +@Experimental public class ListRegionMappingCommand implements GfshCommand { - + static final String EXPERIMENTAL = "(Experimental) "; static final String LIST_MAPPING = "list jdbc-mappings"; - static final String LIST_MAPPING__HELP = "Display jdbc mappings for all members."; + static final String LIST_MAPPING__HELP = EXPERIMENTAL + "Display jdbc mappings for all members."; static final String LIST_OF_MAPPINGS = "List of mappings"; static final String NO_MAPPINGS_FOUND = "No mappings found"; diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListRegionMappingFunction.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListRegionMappingFunction.java index 1e16b41..80e4284 100644 --- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListRegionMappingFunction.java +++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/ListRegionMappingFunction.java @@ -16,10 +16,12 @@ package org.apache.geode.connectors.jdbc.internal.cli; import java.util.Set; +import org.apache.geode.annotations.Experimental; import org.apache.geode.cache.execute.FunctionContext; import org.apache.geode.connectors.jdbc.internal.InternalJdbcConnectorService; import org.apache.geode.connectors.jdbc.internal.RegionMapping; +@Experimental public class ListRegionMappingFunction extends JdbcCliFunction<Void, RegionMapping[]> { ListRegionMappingFunction() { -- To stop receiving notification emails like this one, please contact ['"commits@geode.apache.org" <commits@geode.apache.org>'].