diff --git a/controller-client/src/main/java/org/jboss/as/controller/client/ModelControllerClientConfiguration.java b/controller-client/src/main/java/org/jboss/as/controller/client/ModelControllerClientConfiguration.java index a207e82224d..ef0f4bf3c1f 100644 --- a/controller-client/src/main/java/org/jboss/as/controller/client/ModelControllerClientConfiguration.java +++ b/controller-client/src/main/java/org/jboss/as/controller/client/ModelControllerClientConfiguration.java @@ -80,7 +80,7 @@ public interface ModelControllerClientConfiguration extends Closeable { * @return the SSLContext. * @deprecated Use {@link ModelControllerClientConfiguration#getSslContextFactory()} */ - @Deprecated + @Deprecated(forRemoval = true) SSLContext getSSLContext(); diff --git a/controller/src/main/java/org/jboss/as/controller/AbstractControllerService.java b/controller/src/main/java/org/jboss/as/controller/AbstractControllerService.java index 3e8e400f85c..c93b10c9e24 100644 --- a/controller/src/main/java/org/jboss/as/controller/AbstractControllerService.java +++ b/controller/src/main/java/org/jboss/as/controller/AbstractControllerService.java @@ -191,7 +191,7 @@ private static int getBootStackSize() { * @param authorizer handles authorization * @param capabilityRegistry the capability registry */ - @Deprecated + @Deprecated(forRemoval = true) protected AbstractControllerService(final ProcessType processType, final RunningModeControl runningModeControl, final ConfigurationPersister configurationPersister, final ControlledProcessState processState, final ResourceDefinition rootResourceDefinition, @@ -217,7 +217,7 @@ protected AbstractControllerService(final ProcessType processType, final Running * @param capabilityRegistry the capability registry * @deprecated For use by legacy versions */ - @Deprecated + @Deprecated(forRemoval = true) protected AbstractControllerService(final Supplier executorService, final Supplier instabilityListener, final ProcessType processType, final RunningModeControl runningModeControl, diff --git a/controller/src/main/java/org/jboss/as/controller/AccessAuditContext.java b/controller/src/main/java/org/jboss/as/controller/AccessAuditContext.java index 10df9199bba..75451823561 100644 --- a/controller/src/main/java/org/jboss/as/controller/AccessAuditContext.java +++ b/controller/src/main/java/org/jboss/as/controller/AccessAuditContext.java @@ -131,7 +131,7 @@ public void setDomainRollout(boolean domainRollout) { * @return The current {@link AccessAuditContext} * @deprecated Internal use, will be changed without warning at any time. */ - @Deprecated + @Deprecated(forRemoval = false) public static AccessAuditContext currentAccessAuditContext() { if (WildFlySecurityManager.isChecking()) { System.getSecurityManager().checkPermission(ControllerPermission.GET_CURRENT_ACCESS_AUDIT_CONTEXT); diff --git a/controller/src/main/java/org/jboss/as/controller/CapabilityReferenceRecorder.java b/controller/src/main/java/org/jboss/as/controller/CapabilityReferenceRecorder.java index 65be8fa648d..7d440d8d3d7 100644 --- a/controller/src/main/java/org/jboss/as/controller/CapabilityReferenceRecorder.java +++ b/controller/src/main/java/org/jboss/as/controller/CapabilityReferenceRecorder.java @@ -46,7 +46,7 @@ public interface CapabilityReferenceRecorder extends Feature { * @return base name of dependant, usually name of the attribute that provides reference to capability * @deprecated No longer required and may throw {@link java.lang.UnsupportedOperationException} */ - @Deprecated + @Deprecated(forRemoval = true) String getBaseDependentName(); /** @@ -59,7 +59,7 @@ public interface CapabilityReferenceRecorder extends Feature { * dependent attribute to construct name of capability * @deprecated No longer required and may throw {@link java.lang.UnsupportedOperationException} */ - @Deprecated + @Deprecated(forRemoval = true) default boolean isDynamicDependent() { throw new UnsupportedOperationException(); } diff --git a/controller/src/main/java/org/jboss/as/controller/CompositeOperationHandler.java b/controller/src/main/java/org/jboss/as/controller/CompositeOperationHandler.java index c455dcfdef1..5826f3c9d1e 100644 --- a/controller/src/main/java/org/jboss/as/controller/CompositeOperationHandler.java +++ b/controller/src/main/java/org/jboss/as/controller/CompositeOperationHandler.java @@ -31,7 +31,7 @@ */ public class CompositeOperationHandler implements OperationStepHandler { - @Deprecated + /** Key that domain mode handlers can use to communicate that domain-specific execution has occurred. */ public static final OperationContext.AttachmentKey DOMAIN_EXECUTION_KEY = OperationContext.AttachmentKey.create(Boolean.class); public static final CompositeOperationHandler INSTANCE = new CompositeOperationHandler(); diff --git a/controller/src/main/java/org/jboss/as/controller/ControlledProcessStateService.java b/controller/src/main/java/org/jboss/as/controller/ControlledProcessStateService.java index 371a1cc2623..76e9c65f74a 100644 --- a/controller/src/main/java/org/jboss/as/controller/ControlledProcessStateService.java +++ b/controller/src/main/java/org/jboss/as/controller/ControlledProcessStateService.java @@ -24,7 +24,7 @@ public class ControlledProcessStateService implements ProcessStateNotifier { /** @deprecated use the 'org.wildfly.management.process-state-notifier' capability to obtain a {@link ProcessStateNotifier}*/ - @Deprecated + @Deprecated(forRemoval = true) public static final ServiceName SERVICE_NAME = ServiceName.JBOSS.append("controlled-process-state"); /** Only for use within the WildFly Core kernel; may change or be removed at any time */ public static final ServiceName INTERNAL_SERVICE_NAME = AbstractControllerService.PROCESS_STATE_NOTIFIER_CAPABILITY.getCapabilityServiceName(); diff --git a/controller/src/main/java/org/jboss/as/controller/CurrentOperationIdHolder.java b/controller/src/main/java/org/jboss/as/controller/CurrentOperationIdHolder.java index 4b56724bbd9..de221adc648 100644 --- a/controller/src/main/java/org/jboss/as/controller/CurrentOperationIdHolder.java +++ b/controller/src/main/java/org/jboss/as/controller/CurrentOperationIdHolder.java @@ -12,7 +12,7 @@ * @deprecated internal usage only */ @SuppressWarnings("DeprecatedIsStillUsed") -@Deprecated +@Deprecated(forRemoval = false) public final class CurrentOperationIdHolder { private static final ThreadLocal currentOperationID = new ThreadLocal<>(); diff --git a/controller/src/main/java/org/jboss/as/controller/ListAttributeDefinition.java b/controller/src/main/java/org/jboss/as/controller/ListAttributeDefinition.java index 863889a08be..5a48111a40f 100644 --- a/controller/src/main/java/org/jboss/as/controller/ListAttributeDefinition.java +++ b/controller/src/main/java/org/jboss/as/controller/ListAttributeDefinition.java @@ -75,7 +75,7 @@ public AttributeDefinition getValueAttributeDefinition() { * @throws XMLStreamException if {@code value} is not valid * @deprecated use {@link #getParser()} */ - @Deprecated + @Deprecated(forRemoval = true) public void parseAndAddParameterElement(final String value, final ModelNode operation, final XMLStreamReader reader) throws XMLStreamException { ModelNode paramVal = AttributeParser.SIMPLE.parse(this, value, reader); operation.get(getName()).add(paramVal); diff --git a/controller/src/main/java/org/jboss/as/controller/PersistentResourceXMLParser.java b/controller/src/main/java/org/jboss/as/controller/PersistentResourceXMLParser.java index 900032d802f..e7ffabac1d9 100644 --- a/controller/src/main/java/org/jboss/as/controller/PersistentResourceXMLParser.java +++ b/controller/src/main/java/org/jboss/as/controller/PersistentResourceXMLParser.java @@ -29,7 +29,8 @@ public abstract class PersistentResourceXMLParser implements XMLStreamConstants, public abstract PersistentResourceXMLDescription getParserDescription(); /** @deprecated Experimental; for internal use only. May be removed at any time. */ - @Deprecated + @SuppressWarnings("DeprecatedIsStillUsed") + @Deprecated(forRemoval = false) public final void cacheXMLDescription() { this.cachedDescription.updateAndGet(this); } diff --git a/controller/src/main/java/org/jboss/as/controller/SimpleResourceDefinition.java b/controller/src/main/java/org/jboss/as/controller/SimpleResourceDefinition.java index 08cf60dda59..6c5834403fb 100644 --- a/controller/src/main/java/org/jboss/as/controller/SimpleResourceDefinition.java +++ b/controller/src/main/java/org/jboss/as/controller/SimpleResourceDefinition.java @@ -249,7 +249,7 @@ protected void registerRemoveOperation(final ManagementResourceRegistration regi * @param flags with flags * @deprecated Redundant with {@link #registerAddOperation(ManagementResourceRegistration, OperationStepHandler, org.jboss.as.controller.registry.OperationEntry.Flag...) */ - @Deprecated + @Deprecated(forRemoval = true) protected void registerAddOperation(final ManagementResourceRegistration registration, final AbstractAddStepHandler handler, OperationEntry.Flag... flags) { this.registerAddOperation(registration, (OperationStepHandler) handler, flags); } @@ -262,7 +262,7 @@ protected void registerAddOperation(final ManagementResourceRegistration registr * @param flags with flags * @deprecated Redundant with {@link #registerRemoveOperation(ManagementResourceRegistration, OperationStepHandler, org.jboss.as.controller.registry.OperationEntry.Flag...) */ - @Deprecated + @Deprecated(forRemoval = true) protected void registerRemoveOperation(final ManagementResourceRegistration registration, final AbstractRemoveStepHandler handler, OperationEntry.Flag... flags) { registerRemoveOperation(registration, (OperationStepHandler) handler, flags); } diff --git a/controller/src/main/java/org/jboss/as/controller/descriptions/NonResolvingResourceDescriptionResolver.java b/controller/src/main/java/org/jboss/as/controller/descriptions/NonResolvingResourceDescriptionResolver.java index c9eafeccbd2..6e47ca46244 100644 --- a/controller/src/main/java/org/jboss/as/controller/descriptions/NonResolvingResourceDescriptionResolver.java +++ b/controller/src/main/java/org/jboss/as/controller/descriptions/NonResolvingResourceDescriptionResolver.java @@ -22,7 +22,7 @@ public class NonResolvingResourceDescriptionResolver extends StandardResourceDes * No-arg constructor. * @deprecated use {@link #INSTANCE} instead */ - @Deprecated + @Deprecated(forRemoval = true) public NonResolvingResourceDescriptionResolver() { super("", "", NonResolvingResourceDescriptionResolver.class.getClassLoader()); } diff --git a/controller/src/main/java/org/jboss/as/controller/extension/ExtensionContextSupplement.java b/controller/src/main/java/org/jboss/as/controller/extension/ExtensionContextSupplement.java index 9c1adc91c13..3f6e1689158 100644 --- a/controller/src/main/java/org/jboss/as/controller/extension/ExtensionContextSupplement.java +++ b/controller/src/main/java/org/jboss/as/controller/extension/ExtensionContextSupplement.java @@ -18,7 +18,8 @@ * * @deprecated for internal use only; may be removed at any time */ -@Deprecated +@SuppressWarnings("DeprecatedIsStillUsed") +@Deprecated(forRemoval = false) public interface ExtensionContextSupplement { /** @throws java.lang.UnsupportedOperationException if called from an invalid caller */ AuditLogger getAuditLogger(boolean inheritConfiguration, boolean manualCommit); diff --git a/controller/src/main/java/org/jboss/as/controller/extension/ExtensionRegistry.java b/controller/src/main/java/org/jboss/as/controller/extension/ExtensionRegistry.java index f0e5385965f..9c01eaa60d5 100644 --- a/controller/src/main/java/org/jboss/as/controller/extension/ExtensionRegistry.java +++ b/controller/src/main/java/org/jboss/as/controller/extension/ExtensionRegistry.java @@ -256,7 +256,7 @@ private ExtensionRegistry(Builder builder) { * @param hostControllerInfoAccessor the host controller * @deprecated Use {@link #builder(ProcessType)} instead. */ - @Deprecated + @Deprecated(forRemoval = true) public ExtensionRegistry(ProcessType processType, RunningModeControl runningModeControl, ManagedAuditLogger auditLogger, JmxAuthorizer authorizer, Supplier securityIdentitySupplier, RuntimeHostControllerInfoAccessor hostControllerInfoAccessor) { this(builder(processType).withRunningModeControl(runningModeControl) @@ -273,7 +273,7 @@ public ExtensionRegistry(ProcessType processType, RunningModeControl runningMode * @param runningModeControl the process' running mode * @deprecated Here for core-model-test and subsystem-test backwards compatibility */ - @Deprecated + @Deprecated(forRemoval = true) public ExtensionRegistry(ProcessType processType, RunningModeControl runningModeControl) { this(builder(processType).withRunningModeControl(runningModeControl)); } diff --git a/controller/src/main/java/org/jboss/as/controller/remote/RemoteProxyController.java b/controller/src/main/java/org/jboss/as/controller/remote/RemoteProxyController.java index 99f5850f466..2f3a450e779 100644 --- a/controller/src/main/java/org/jboss/as/controller/remote/RemoteProxyController.java +++ b/controller/src/main/java/org/jboss/as/controller/remote/RemoteProxyController.java @@ -78,7 +78,7 @@ public static RemoteProxyController create(final TransactionalProtocolClient cli * * @deprecated only present for test case use */ - @Deprecated + @Deprecated(forRemoval = false) public static RemoteProxyController create(final ManagementChannelHandler channelAssociation, final PathAddress pathAddress, final ProxyOperationAddressTranslator addressTranslator) { final TransactionalProtocolClient client = TransactionalProtocolHandlers.createClient(channelAssociation); // the remote proxy diff --git a/controller/src/main/java/org/jboss/as/controller/services/path/PathManagerService.java b/controller/src/main/java/org/jboss/as/controller/services/path/PathManagerService.java index 53585a661b5..c15f55fe511 100644 --- a/controller/src/main/java/org/jboss/as/controller/services/path/PathManagerService.java +++ b/controller/src/main/java/org/jboss/as/controller/services/path/PathManagerService.java @@ -44,8 +44,8 @@ */ public abstract class PathManagerService implements PathManager, Service { - /** @deprecated ServiceName should be obtained from capability */ - @Deprecated + /** @deprecated ServiceName should be obtained from capability 'org.wildfly.management.path-manager'.*/ + @Deprecated(forRemoval = true) public static final ServiceName SERVICE_NAME = ServiceName.JBOSS.append("path", "manager"); //@GuardedBy(pathEntries) diff --git a/controller/src/main/java/org/jboss/as/controller/services/path/ResolvePathHandler.java b/controller/src/main/java/org/jboss/as/controller/services/path/ResolvePathHandler.java index 9b14216120b..0d4b77335c7 100644 --- a/controller/src/main/java/org/jboss/as/controller/services/path/ResolvePathHandler.java +++ b/controller/src/main/java/org/jboss/as/controller/services/path/ResolvePathHandler.java @@ -170,7 +170,7 @@ public static Builder of(final PathManager pathManager) { * * @return the operation handler builder */ - @Deprecated + @Deprecated(forRemoval = false) public static Builder of(final String operationName, final PathManager pathManager) { Assert.checkNotNullParam("operationName", operationName); Assert.checkNotNullParam("pathManager", pathManager); diff --git a/controller/src/main/java/org/jboss/as/controller/transform/AliasOperationTransformer.java b/controller/src/main/java/org/jboss/as/controller/transform/AliasOperationTransformer.java index c707d2dbb3b..10c43c897bc 100644 --- a/controller/src/main/java/org/jboss/as/controller/transform/AliasOperationTransformer.java +++ b/controller/src/main/java/org/jboss/as/controller/transform/AliasOperationTransformer.java @@ -17,7 +17,6 @@ * * @author Emanuel Muckenhuber */ -@Deprecated //todo could probably be removed as it is not used anywhere public class AliasOperationTransformer implements CombinedTransformer { public interface AddressTransformer { diff --git a/controller/src/main/java/org/jboss/as/controller/transform/ResourceTransformationContextImpl.java b/controller/src/main/java/org/jboss/as/controller/transform/ResourceTransformationContextImpl.java index f49b53a657c..fda5b75d3d4 100644 --- a/controller/src/main/java/org/jboss/as/controller/transform/ResourceTransformationContextImpl.java +++ b/controller/src/main/java/org/jboss/as/controller/transform/ResourceTransformationContextImpl.java @@ -322,7 +322,6 @@ public Resource getTransformedRoot() { return root; } - @Deprecated static ResourceTransformationContext createAliasContext(final PathAddress address, final ResourceTransformationContext context) { if (context instanceof ResourceTransformationContextImpl) { final ResourceTransformationContextImpl impl = (ResourceTransformationContextImpl) context; @@ -333,7 +332,6 @@ static ResourceTransformationContext createAliasContext(final PathAddress addres } } - @Deprecated static TransformationContext wrapForOperation(TransformationContext context, ModelNode operation) { if(context instanceof ResourceTransformationContextImpl) { final ResourceTransformationContextImpl impl = (ResourceTransformationContextImpl) context; diff --git a/controller/src/main/java/org/jboss/as/controller/transform/description/RejectedAttributesLogContext.java b/controller/src/main/java/org/jboss/as/controller/transform/description/RejectedAttributesLogContext.java index ccba48027fc..0f43840fed2 100644 --- a/controller/src/main/java/org/jboss/as/controller/transform/description/RejectedAttributesLogContext.java +++ b/controller/src/main/java/org/jboss/as/controller/transform/description/RejectedAttributesLogContext.java @@ -116,7 +116,7 @@ String errorOrWarnOnResourceTransformation() throws OperationFailedException { return null; } - @Deprecated //todo replace with context.getLogger().... + //todo replace with context.getLogger().... String getOperationRejectDescription() { if (op == null) { throw new IllegalStateException(); diff --git a/server/src/main/java/org/jboss/as/server/ExternalManagementRequestExecutor.java b/server/src/main/java/org/jboss/as/server/ExternalManagementRequestExecutor.java index 0bc643e40a8..22a1bebfe5b 100644 --- a/server/src/main/java/org/jboss/as/server/ExternalManagementRequestExecutor.java +++ b/server/src/main/java/org/jboss/as/server/ExternalManagementRequestExecutor.java @@ -36,7 +36,7 @@ * * @author Brian Stansberry */ -@Deprecated +@Deprecated(forRemoval = false) public class ExternalManagementRequestExecutor implements Service { /** @@ -44,7 +44,7 @@ public class ExternalManagementRequestExecutor implements Service ServiceBuilder addServerExecutorDependency(ServiceBuilder builder, Injector injector) { return builder.addDependency(ServerService.MANAGEMENT_EXECUTOR, ExecutorService.class, injector); } diff --git a/server/src/main/java/org/jboss/as/server/controller/resources/ServerRootResourceDefinition.java b/server/src/main/java/org/jboss/as/server/controller/resources/ServerRootResourceDefinition.java index 1748a356f69..36874c0a713 100644 --- a/server/src/main/java/org/jboss/as/server/controller/resources/ServerRootResourceDefinition.java +++ b/server/src/main/java/org/jboss/as/server/controller/resources/ServerRootResourceDefinition.java @@ -231,7 +231,7 @@ public class ServerRootResourceDefinition extends SimpleResourceDefinition { * The 'timeout' parameter for server lifecycle ops * @deprecated Since Version 9.0.0, use suspend-timeout instead. */ - @Deprecated + @Deprecated(forRemoval = false) // false because this @Deprected is just to help us remember to remove this some day. We don't want IDE errors about it. public static final SimpleAttributeDefinition TIMEOUT = new SimpleAttributeDefinitionBuilder(ModelDescriptionConstants.TIMEOUT, ModelType.INT) .setDefaultValue(ModelNode.ZERO) .setRequired(false) diff --git a/server/src/main/java/org/jboss/as/server/deployment/ContentCleanerService.java b/server/src/main/java/org/jboss/as/server/deployment/ContentCleanerService.java index 5ab8993d916..fa2049e8b27 100644 --- a/server/src/main/java/org/jboss/as/server/deployment/ContentCleanerService.java +++ b/server/src/main/java/org/jboss/as/server/deployment/ContentCleanerService.java @@ -38,7 +38,8 @@ public class ContentCleanerService implements Service { * * @deprecated DON'T USE IT. */ - @Deprecated + @SuppressWarnings("DeprecatedIsStillUsed") + @Deprecated(forRemoval = true) private static final String UNSUPPORTED_PROPERTY = "org.wildfly.unsupported.content.repository.obsolescence"; /** * The conten repository cleaner will test content for clean-up every 5 minutes. diff --git a/server/src/main/java/org/jboss/as/server/deployment/DeploymentModelUtils.java b/server/src/main/java/org/jboss/as/server/deployment/DeploymentModelUtils.java index 07afad1a545..a8d4e1ddfd2 100644 --- a/server/src/main/java/org/jboss/as/server/deployment/DeploymentModelUtils.java +++ b/server/src/main/java/org/jboss/as/server/deployment/DeploymentModelUtils.java @@ -12,7 +12,7 @@ * @author Emanuel Muckenhuber * @deprecated Use {@link org.jboss.as.server.deployment.DeploymentResourceSupport} from an {@link Attachments#DEPLOYMENT_RESOURCE_SUPPORT attachment} on the {@link org.jboss.as.server.deployment.DeploymentUnit} */ -@Deprecated +@Deprecated(forRemoval = true) public class DeploymentModelUtils { public static final AttachmentKey DEPLOYMENT_RESOURCE = DeploymentResourceSupport.DEPLOYMENT_RESOURCE; diff --git a/server/src/main/java/org/jboss/as/server/deployment/client/ModelControllerServerDeploymentManager.java b/server/src/main/java/org/jboss/as/server/deployment/client/ModelControllerServerDeploymentManager.java index c4444ac868d..5ae9ff47825 100644 --- a/server/src/main/java/org/jboss/as/server/deployment/client/ModelControllerServerDeploymentManager.java +++ b/server/src/main/java/org/jboss/as/server/deployment/client/ModelControllerServerDeploymentManager.java @@ -20,7 +20,7 @@ * * @deprecated Use {@link org.jboss.as.controller.client.helpers.standalone.ServerDeploymentManager.Factory} */ -@Deprecated +@Deprecated(forRemoval = true) public class ModelControllerServerDeploymentManager extends AbstractServerDeploymentManager { private final ModelControllerClient client; diff --git a/server/src/main/java/org/jboss/as/server/deployment/transformation/DeploymentTransformer.java b/server/src/main/java/org/jboss/as/server/deployment/transformation/DeploymentTransformer.java index 76257b2cc1f..180e63ad4d6 100644 --- a/server/src/main/java/org/jboss/as/server/deployment/transformation/DeploymentTransformer.java +++ b/server/src/main/java/org/jboss/as/server/deployment/transformation/DeploymentTransformer.java @@ -12,10 +12,11 @@ /** * Performs transformation operations on deployment content. - * + *

* This interface is experimental and may be removed or altered at any time. */ -@Deprecated +@SuppressWarnings("DeprecatedIsStillUsed") +@Deprecated(forRemoval = false) public interface DeploymentTransformer { /** diff --git a/server/src/main/java/org/jboss/as/server/services/net/NetworkInterfaceService.java b/server/src/main/java/org/jboss/as/server/services/net/NetworkInterfaceService.java index 550efd51c63..c3f7492233b 100644 --- a/server/src/main/java/org/jboss/as/server/services/net/NetworkInterfaceService.java +++ b/server/src/main/java/org/jboss/as/server/services/net/NetworkInterfaceService.java @@ -40,14 +40,14 @@ */ public class NetworkInterfaceService implements Service { - private static ServerLogger log = ServerLogger.NETWORK_LOGGER; + private static final ServerLogger log = ServerLogger.NETWORK_LOGGER; /** * The service base name. * * @deprecated use the org.wildfly.network.interface capability */ - @Deprecated + @Deprecated(forRemoval = true) public static final ServiceName JBOSS_NETWORK_INTERFACE = ServiceName.JBOSS.append("network"); private static final String IPV4_ANYLOCAL = "0.0.0.0"; diff --git a/threads/src/main/java/org/jboss/as/threads/BoundedQueueThreadPoolResourceDefinition.java b/threads/src/main/java/org/jboss/as/threads/BoundedQueueThreadPoolResourceDefinition.java index d1d5ab5f662..d99848c898d 100644 --- a/threads/src/main/java/org/jboss/as/threads/BoundedQueueThreadPoolResourceDefinition.java +++ b/threads/src/main/java/org/jboss/as/threads/BoundedQueueThreadPoolResourceDefinition.java @@ -79,7 +79,7 @@ public static BoundedQueueThreadPoolResourceDefinition create(PathElement path, /** * @deprecated This class is not designed for subclassing and having this constructor be accessible is a specific workaround for WFCORE-1623 that may be reverted at any time */ - @Deprecated + @Deprecated(forRemoval = false) protected BoundedQueueThreadPoolResourceDefinition(boolean blocking, boolean registerRuntimeOnly, String type, ServiceName serviceNameBase, String resolverPrefix, OperationStepHandler addHandler, OperationStepHandler removeHandler) { diff --git a/threads/src/main/java/org/jboss/as/threads/ThreadSubsystemResourceDefinition.java b/threads/src/main/java/org/jboss/as/threads/ThreadSubsystemResourceDefinition.java index afdb5a068c9..328bd8c2b0c 100644 --- a/threads/src/main/java/org/jboss/as/threads/ThreadSubsystemResourceDefinition.java +++ b/threads/src/main/java/org/jboss/as/threads/ThreadSubsystemResourceDefinition.java @@ -21,13 +21,12 @@ * * @author Brian Stansberry (c) 2011 Red Hat Inc. */ -@Deprecated -@SuppressWarnings("deprecation") +@Deprecated(forRemoval = true) +@SuppressWarnings({"removal", "DeprecatedIsStillUsed"}) class ThreadSubsystemResourceDefinition extends PersistentResourceDefinition { private final boolean registerRuntimeOnly; ThreadSubsystemResourceDefinition(boolean runtimeOnly) { - //noinspection deprecation super(ThreadsExtension.SUBSYSTEM_PATH, new DeprecatedResourceDescriptionResolver(ThreadsExtension.SUBSYSTEM_NAME, ThreadsExtension.SUBSYSTEM_NAME, ThreadsExtension.RESOURCE_NAME, ThreadsExtension.class.getClassLoader(), true, false), ThreadsSubsystemAdd.INSTANCE, ReloadRequiredRemoveStepHandler.INSTANCE, diff --git a/threads/src/main/java/org/jboss/as/threads/ThreadsExtension.java b/threads/src/main/java/org/jboss/as/threads/ThreadsExtension.java index fc271c39b77..54676b7e6d7 100644 --- a/threads/src/main/java/org/jboss/as/threads/ThreadsExtension.java +++ b/threads/src/main/java/org/jboss/as/threads/ThreadsExtension.java @@ -23,7 +23,10 @@ * * @author David M. Lloyd * @author Kabir Khan + * + * @deprecated the code module containing this extension only exists to support use of particular resource definition classes in other subsystems. */ +@Deprecated(forRemoval = true) public class ThreadsExtension extends AbstractLegacyExtension { public static final String SUBSYSTEM_NAME = "threads"; @@ -52,7 +55,7 @@ protected Set initializeLegacyModel(ExtensionCon registration.registerXMLElementWriter(ThreadsParser2_0::new); // Remoting threads description and operation handlers - @SuppressWarnings("deprecation") + @SuppressWarnings({"removal"}) final ManagementResourceRegistration subsystem = registration.registerSubsystemModel(new ThreadSubsystemResourceDefinition(registerRuntimeOnly)); return Collections.singleton(subsystem); diff --git a/threads/src/main/java/org/jboss/as/threads/UnboundedQueueThreadPoolWriteAttributeHandler.java b/threads/src/main/java/org/jboss/as/threads/UnboundedQueueThreadPoolWriteAttributeHandler.java index cbdfe11e65b..944375f674e 100644 --- a/threads/src/main/java/org/jboss/as/threads/UnboundedQueueThreadPoolWriteAttributeHandler.java +++ b/threads/src/main/java/org/jboss/as/threads/UnboundedQueueThreadPoolWriteAttributeHandler.java @@ -26,7 +26,7 @@ public class UnboundedQueueThreadPoolWriteAttributeHandler extends ThreadsWriteA private final ServiceName serviceNameBase; private final RuntimeCapability capability; - @Deprecated + @Deprecated(forRemoval = true) public UnboundedQueueThreadPoolWriteAttributeHandler(ServiceName serviceNameBase) { this(null, serviceNameBase); }