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

Add support for FT.ALTER #28

Merged
merged 57 commits into from
May 1, 2020
Merged

Add support for FT.ALTER #28

merged 57 commits into from
May 1, 2020

Conversation

felixls
Copy link
Contributor

@felixls felixls commented Nov 28, 2019

#25

@codecov
Copy link

codecov bot commented Nov 28, 2019

Codecov Report

Merging #28 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #28   +/-   ##
=======================================
  Coverage   75.04%   75.04%           
=======================================
  Files          12       12           
  Lines        1182     1182           
=======================================
  Hits          887      887           
  Misses        231      231           
  Partials       64       64           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a6e496...4a6e496. Read the comment docs.

耿斌 and others added 2 commits December 5, 2019 16:22
[fix] the parameters LANGUAGE is not supported in the FT.SEARCH
redisearch/client.go Show resolved Hide resolved
redisearch/client.go Outdated Show resolved Hide resolved
filipecosta90 and others added 23 commits April 1, 2020 18:50
…ncluded to make code more readable -- no breaking changes)
support for FT.GET, FT.MGET, FT.SUGLEN, FT.SUGDEL, FT.ALIASADD, FT.ALIASUPDATE, FT.ALIASDEL, FT.DICTADD, FT.DICTDEL, FT.DICTDUMP
Support FT.CONFIG command
* [add] added option to create clients with shared pool. added option to releases the resources used by the pool ( single, and multihost )

* [add] added Close() test on multi-host pool

* [add] extended test for Close on multihost pool

* [add] swapped deprecated redis.NewPool to redis.Pool constructor. preserving all errors on multihost pool CLose()

* [fix] fixed CI invalid memory address or nil pointer dereference on TestNewMultiHostPool/multi-host_single_address
* Support FT.SYNADD FT.SYNUPDATE FT.SYNDUMP commands
* Support FT.ADDHASH command
* NewSchema fix and extended testing for SerializeSchema
@filipecosta90 filipecosta90 self-requested a review May 1, 2020 01:25
Copy link
Collaborator

@filipecosta90 filipecosta90 left a comment

Choose a reason for hiding this comment

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

approved as per the latest rebase from master and minor fix

@filipecosta90 filipecosta90 merged commit 8eec4dd into RediSearch:master May 1, 2020
@filipecosta90 filipecosta90 changed the title Add schema add and MAXTEXTFIELD support Add support for FT.ALTER Jul 26, 2020
@filipecosta90
Copy link
Collaborator

we've correct the PR name from Add schema add and MAXTEXTFIELD support to Add support for FT.ALTER given that this PR only focus on the addition of FT.ALTER command.

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.

4 participants