Skip to content

DOC-5471 time series doc examples #4210

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

Merged

Conversation

andy-stark-redis
Copy link
Contributor

@andy-stark-redis andy-stark-redis commented Jul 24, 2025

DOC-5471

Jedis examples to match the Python code in the time series doc page.

A slight issue is that the MAdd calls produce a warning:

Type safety: A generic array of AbstractMap.SimpleEntry<> is created for a varargs parameter

This doesn't stop compilation and what I can find with Google about it suggests it's not a big problem. But it there's a way to avoid this then I'll certainly add it.

@andy-stark-redis andy-stark-redis marked this pull request as ready for review July 24, 2025 11:41
@ggivo ggivo added this to the 6.1.0 milestone Jul 25, 2025
Copy link
Collaborator

@ggivo ggivo left a comment

Choose a reason for hiding this comment

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

LGTM

@ggivo ggivo merged commit 1ecb037 into redis:master Jul 25, 2025
9 of 14 checks passed
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.

2 participants