[FIXED] MQTT: more generic names for outgoing stream, etc. #4484
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
git pull --rebase origin dev
)Changes proposed in this pull request:
Removed
qos2
reference from the MQTT outgoing stream name and subjects.Rationale: As I was reviewing the documentation and thinking how to move forward with MQTT, I am inclined to use the
$MQTT_out
stream and session-specific consumers (as opposed to subscription-specific) for all QOS-related packet delivery and PI persistence. I think I will be able to repurpose it "as is", so proposing the generic name change.Since these (stream, consumer, subscriptions) are newly introduced in #4349 and have not yet been released, this is a safe change.