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

Use FTCreateParams for ftCreate method #357

Closed
wants to merge 8 commits into from

Conversation

sazzad16
Copy link
Contributor

@sazzad16 sazzad16 commented Oct 9, 2023

To do:

  • Access 'name' of SchemaField

@sazzad16 sazzad16 requested a review from bsbodden October 9, 2023 11:39
@bsbodden
Copy link
Contributor

@sazzad16 other than missing the SchemaField#name, the changes look good and consistent in Jedis!

sazzad16

This comment was marked as resolved.

sazzad16 added a commit to sazzad16/jedis that referenced this pull request Oct 15, 2023
All I know these changes have been enough to pass all redis-om-spring tests in redis/redis-om-spring#357
sazzad16 added a commit to redis/jedis that referenced this pull request Oct 15, 2023
All I know these changes have been enough to pass all redis-om-spring tests in redis/redis-om-spring#357
sazzad16 added a commit to redis/jedis that referenced this pull request Oct 15, 2023
All I know these changes have been enough to pass all redis-om-spring tests in redis/redis-om-spring#357
sazzad16 added a commit to redis/jedis that referenced this pull request Oct 15, 2023
All I know these changes have been enough to pass all redis-om-spring tests in redis/redis-om-spring#357
It has been confirmed by RediSearch developers that it has no effect.
It is likely to be residue from older versions.
bsbodden
bsbodden previously approved these changes Oct 19, 2023
Copy link
Contributor

@bsbodden bsbodden left a comment

Choose a reason for hiding this comment

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

This is perfect, thanks @sazzad16 now we just need to wait for Spring to up the Jedis version - I'll merge this when they do!

@bsbodden
Copy link
Contributor

bsbodden commented Jan 1, 2024

Cherry-picked on separate PR

@bsbodden bsbodden closed this Jan 1, 2024
@sazzad16 sazzad16 deleted the ft-create-params branch January 2, 2024 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants