[ https://issues.apache.org/jira/browse/AMQ-7514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17377864#comment-17377864 ]
Étienne Hossack edited comment on AMQ-7514 at 7/9/21, 7:06 AM: --------------------------------------------------------------- Regarding your concerns [~michael.andre.pearce] for ActiveMQ 5.17+ as those are the ones currently in question, since leveldb has been removed, I don't think we have to worry about that confusion! Yes it could change in the future, but the discussion is for right now, and we can't really predict the future it's a good path forward in my opinion. For Artemis I hear your concerns about states and designations. So rather than introduce something new, maybe keeping the current live/backup as is used in the docs is best? The config can reference Active/Replica or whichever as needed to determine the intent of the node. I'll reiterate that "primary" or "leader" are moderately confusing in the context of multiple HA pairs within an Artemis cluster. I don't believe any of the terms in the linked article are any clearer. Your sentence "When an Active node fails the Replica takes over, but if you have fail back to Active configured when the Active node is restarted the Active Replica will failback to the Active node and it will become Active again" can be written as: "When an Active node fails the Replica takes over [accepting traffic]. If you have configured your broker with the 'allow-failback' option, when the original Active is restarted, the Replica will resume replication and the Active node will accept traffic." I believe the confusion in your sentence comes from the capitalization and referencing "failback" as concept before defining what that actually means in the context of this HA pair. In general I believe a broker user/documentation reader does not need to be aware of a formal state in this context, and the implicit meaning in a sentence can suffice (serving traffic). edit: It might even reduce the cognitive load, since it's one less concept to have to learn. was (Author: ehossack): Regarding your concerns [~michael.andre.pearce] for ActiveMQ 5.17+ as those are the ones currently in question, since leveldb has been removed, I don't think we have to worry about that confusion! Yes it could change in the future, but the discussion is for right now, and we can't really predict the future it's a good path forward in my opinion. For Artemis I hear your concerns about states and designations. So rather than introduce something new, maybe keeping the current live/backup as is used in the docs is best? The config can reference Active/Replica or whichever as needed to determine the intent of the node. I'll reiterate that "primary" or "leader" are moderately confusing in the context of multiple HA pairs within an Artemis cluster. I don't believe any of the terms in the linked article are any clearer. Your sentence "When an Active node fails the Replica takes over, but if you have fail back to Active configured when the Active node is restarted the Active Replica will failback to the Active node and it will become Active again" can be written as: "When an Active node fails the Replica takes over [accepting traffic]. If you have configured your broker with the 'allow-failback' option, when the original Active is restarted, the Replica will resume replication and the Active node will accept traffic." I believe the confusion in your sentence comes from the capitalization and referencing "failback" as concept before defining what that actually means in the context of this HA pair. In general I believe a broker user/documentation reader does not need to be aware of a formal state in this context, and the implicit meaning in a sentence can suffice (serving traffic). > Replace racially charged 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: 3h 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)