You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@activemq.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2021/08/20 20:59:00 UTC

[jira] [Work logged] (AMQ-7514) Replace use of non-inclusive terms throughout source code, comments and documentation

     [ https://issues.apache.org/jira/browse/AMQ-7514?focusedWorklogId=640390&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-640390 ]

ASF GitHub Bot logged work on AMQ-7514:
---------------------------------------

                Author: ASF GitHub Bot
            Created on: 20/Aug/21 20:58
            Start Date: 20/Aug/21 20:58
    Worklog Time Spent: 10m 
      Work Description: ehossack-aws commented on a change in pull request #679:
URL: https://github.com/apache/activemq/pull/679#discussion_r693215288



##########
File path: activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java
##########
@@ -485,15 +490,15 @@ public JmsConnector removeJmsConnector(JmsConnector connector) {
     }
 
     public void masterFailed() {
-        if (shutdownOnMasterFailure) {
-            LOG.error("The Master has failed ... shutting down");
+        if (shutdownOnActiveFailure) {
+            LOG.error("The Active has failed ... shutting down");

Review comment:
       as per @mattrpav 's AMQ-8316, moved the two concretely deprecated messages out of this PR

##########
File path: activemq-broker/src/main/java/org/apache/activemq/broker/BrokerService.java
##########
@@ -485,15 +490,15 @@ public JmsConnector removeJmsConnector(JmsConnector connector) {
     }
 
     public void masterFailed() {
-        if (shutdownOnMasterFailure) {
-            LOG.error("The Master has failed ... shutting down");
+        if (shutdownOnActiveFailure) {
+            LOG.error("The Active has failed ... shutting down");

Review comment:
       as per @mattrpav 's AMQ-8316, moved the two concretely deprecated messages out of this PR into https://github.com/apache/activemq/pull/700




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 640390)
    Time Spent: 4h 50m  (was: 4h 40m)

> Replace use of non-inclusive terms throughout source code, comments and documentation
> -------------------------------------------------------------------------------------
>
>                 Key: AMQ-7514
>                 URL: https://issues.apache.org/jira/browse/AMQ-7514
>             Project: ActiveMQ
>          Issue Type: Task
>            Reporter: Bruce Snyder
>            Priority: Major
>          Time Spent: 4h 50m
>  Remaining Estimate: 0h
>
> Given the racial charged nature of certain terms in today's world, we must pull together to create a plan for changing any such terms throughout all the ActiveMQ projects and in the git repos themselves.
>   
>  Example: [https://activemq.apache.org/masterslave.html]
>   
>  Here are just a few terms that should be changed: * The following terms are being targeted for change:
>  * 
>  ** 'master' and 'slave' should be replaced with the terms 'live' and 'backup'
>  ** 'whitelist' and 'blacklist' should be replaced with the terms 'allowlist' and 'denylist'
>  * Rename all the git 'master' branches to the term 'main'
> Proposal notes from activemq-dev mailing list
> Phase 1: 
> 1. Deprecate terms such as ‘master’ and ’slave
> 2. log.warn any configuration change notifications
> 3. Provide compatibility under the covers for deprecated terms
> 4. Provide any openwire compatibility changes b/w ActiveMQ 5 and Artemis
> 5. Notify users in an announcement and provide a conversion HOWTO
> Phase 2: 
> 1. Remove terminology as part of a major or minor release (SEMVER where ‘y’ in ‘x.y.z’ is minor version number)
> New terms:
> a. For shared storage: ‘active’ and ’standby’
> b. For replication: ‘primary’ and ‘replica'
> c. For 'white list' and 'blacklist': 'allow list' and 'deny list'
> For example:
> ‘master’ -> ‘active’
> ’slave’ -> ’standby'



--
This message was sent by Atlassian Jira
(v8.3.4#803005)