[activemq] branch main updated: [AMQ-8316] Flag deprecated methods in BrokerService (#988)

[cshannon]

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

cshannon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/activemq.git


The following commit(s) were added to refs/heads/main by this push:
     new c41be2a8d [AMQ-8316] Flag deprecated methods in BrokerService (#988)
c41be2a8d is described below

commit c41be2a8d765448b825c50388b1c5c137593194a
Author: JB Onofré <jbono...@apache.org>
AuthorDate: Tue Mar 14 12:07:56 2023 +0100

    [AMQ-8316] Flag deprecated methods in BrokerService (#988)
    
    These methods will be removed or renamed in a future release.
    
    Co-authored-by: Christopher L. Shannon <christopher.l.shan...@gmail.com>
---
 .../org/apache/activemq/broker/BrokerService.java  | 31 +++++++++++++++++++---
 1 file changed, 28 insertions(+), 3 deletions(-)

diff --git 
a/activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java 
b/activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java
index 64858e9f8..8c969571c 100644
--- 
a/activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java
+++ 
b/activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java
@@ -1672,6 +1672,7 @@ public class BrokerService implements Service {
     /**
      * @return Returns the shutdownOnMasterFailure.
      */
+    @Deprecated(forRemoval = true)
     public boolean isShutdownOnMasterFailure() {
         return shutdownOnMasterFailure;
     }
@@ -2901,49 +2902,73 @@ public class BrokerService implements Service {
         this.sslContext = sslContext;
     }
 
+    /**
+     * @deprecated this method will be renamed to not use slave wording
+     */
+    @Deprecated(forRemoval = true)
     public boolean isShutdownOnSlaveFailure() {
         return shutdownOnSlaveFailure;
     }
 
     /**
+     * @deprecated this method will be renamed to not use slave wording
+     *
      * @org.apache.xbean.Property 
propertyEditor="org.apache.activemq.util.BooleanEditor"
      */
+    @Deprecated(forRemoval = true)
     public void setShutdownOnSlaveFailure(boolean shutdownOnSlaveFailure) {
         this.shutdownOnSlaveFailure = shutdownOnSlaveFailure;
     }
 
+    /**
+     * @deprecated it will be removed as it should not be used directly.
+     */
+    @Deprecated(forRemoval = true)
     public boolean isWaitForSlave() {
         return waitForSlave;
     }
 
     /**
+     * @deprecated this method will be renamed to not use slave wording
+     *
      * @org.apache.xbean.Property 
propertyEditor="org.apache.activemq.util.BooleanEditor"
      */
+    @Deprecated(forRemoval = true)
     public void setWaitForSlave(boolean waitForSlave) {
         this.waitForSlave = waitForSlave;
     }
 
+    /**
+     * @deprecated this method will be renamed to not use slave wording
+     */
+    @Deprecated(forRemoval = true)
     public long getWaitForSlaveTimeout() {
         return this.waitForSlaveTimeout;
     }
 
+    /**
+     * @deprecated this method will be renamed to not use slave wording
+     */
+    @Deprecated(forRemoval = true)
     public void setWaitForSlaveTimeout(long waitForSlaveTimeout) {
         this.waitForSlaveTimeout = waitForSlaveTimeout;
     }
 
     /**
-     * Get the passiveSlave
-     * @return the passiveSlave
+     * @deprecated this method will be renamed to not use slave wording
      */
+    @Deprecated(forRemoval = true)
     public boolean isPassiveSlave() {
         return this.passiveSlave;
     }
 
     /**
-     * Set the passiveSlave
+     * @deprecated this method will be renamed to not use slave wording
+     *
      * @param passiveSlave the passiveSlave to set
      * @org.apache.xbean.Property 
propertyEditor="org.apache.activemq.util.BooleanEditor"
      */
+    @Deprecated(forRemoval = true)
     public void setPassiveSlave(boolean passiveSlave) {
         this.passiveSlave = passiveSlave;
     }