Skip to content

Conversation

@enjoy-binbin
Copy link
Contributor

We already have pubsub_channels and pubsub_patterns
in INFO stats, now add pubsubshard_channels (symmetry).

Sharded pubsub was added in #8621

We already have `pubsub_channels` and `pubsub_patterns`
in INFO stats, now add `pubsubshard_channels` (symmetry).

Sharded pubsub was added in redis#8621
@enjoy-binbin enjoy-binbin added the state:needs-doc-pr requires a PR to redis-doc repository label Jul 4, 2022
@oranagra
Copy link
Member

oranagra commented Jul 4, 2022

@itamarhaber do you happen to know what this info is used for (other than maybe testing and debugging)?

@itamarhaber
Copy link
Member

No, but I would assume these can be used for monitoring the deployment's high-level activity. The PUBSUB command provides a finer-grained output of that information.

@oranagra
Copy link
Member

oranagra commented Jul 4, 2022

@redis/core-team please approve new info field or comment.

Copy link
Contributor

@madolson madolson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK!

@madolson madolson added the approval-needed Waiting for core team approval to be merged label Jul 4, 2022
@oranagra oranagra merged commit 0132ed7 into redis:unstable Jul 6, 2022
@enjoy-binbin enjoy-binbin deleted the add_pubsubshard_channels_to_info branch July 6, 2022 06:51
@oranagra oranagra added the release-notes indication that this issue needs to be mentioned in the release notes label Jul 10, 2022
@oranagra oranagra mentioned this pull request Jul 11, 2022
enjoy-binbin added a commit to enjoy-binbin/redis that referenced this pull request Jul 31, 2023
We already have `pubsub_channels` and `pubsub_patterns`
in INFO stats, now add `pubsubshard_channels` (symmetry).

Sharded pubsub was added in redis#8621
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval-needed Waiting for core team approval to be merged release-notes indication that this issue needs to be mentioned in the release notes state:needs-doc-pr requires a PR to redis-doc repository

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants