Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DEV+INT: update INFO command and Prometheus v2 metrics for search #1157

Merged
merged 4 commits into from
Feb 12, 2025

Conversation

dwdougherty
Copy link
Collaborator

@dwdougherty dwdougherty commented Feb 11, 2025

DOC-4611

INFO command page and Prometheus for RS v2 page updates.

@dwdougherty dwdougherty requested review from meiravgri and removed request for maayanagranat February 11, 2025 16:07
Copy link
Collaborator

@rrelledge rrelledge left a comment

Choose a reason for hiding this comment

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

Looks good to me. I just left a few small suggestions to consider.

Copy link

@meiravgri meiravgri left a comment

Choose a reason for hiding this comment

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

Looks good!

  1. Please add the version in which each of metric was introduced.
  2. In the shard section, all Prometheus metrics—including search metrics—are prefixed with redis_server. For example, search_total_query_execution_time_ms would be recorded as redis_server_search_total_query_execution_time_ms

content/commands/info/index.md Outdated Show resolved Hide resolved
@dwdougherty
Copy link
Collaborator Author

Thank you @rrelledge and @meiravgri ! All comments addressed.

@dwdougherty dwdougherty merged commit 065db6d into main Feb 12, 2025
5 checks passed
@dwdougherty dwdougherty deleted the DOC-4611 branch February 12, 2025 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants