Skip to content

Add support for JSON, TIMESERIES, BLOOM & GRAPH commands in cluster #2032

New issue

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

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

Already on GitHub? # to your account

Merged
merged 10 commits into from
Mar 6, 2022

Conversation

dvora-h
Copy link
Collaborator

@dvora-h dvora-h commented Mar 2, 2022

Pull Request check-list

Please make sure to review and check all of these items:

  • Does $ tox pass with this change (including linting)?
  • Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
  • Is the new or changed code fully tested?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
  • Is there an example added to the examples folder (if applicable)?
  • Was the change added to CHANGES file?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.

Description of change

as part of #1982

@dvora-h dvora-h added the feature New feature label Mar 2, 2022
@dvora-h dvora-h changed the title Add support for JSON commands in cluster Add support for JSON, TIMESERIES, BLOOM & GRAPH commands in cluster Mar 4, 2022
@codecov-commenter
Copy link

codecov-commenter commented Mar 6, 2022

Codecov Report

Merging #2032 (6fcfbaf) into master (c5d19b8) will decrease coverage by 0.00%.
The diff coverage is 78.37%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2032      +/-   ##
==========================================
- Coverage   92.63%   92.63%   -0.01%     
==========================================
  Files         100      100              
  Lines       20921    20953      +32     
==========================================
+ Hits        19381    19409      +28     
- Misses       1540     1544       +4     
Impacted Files Coverage Δ
redis/cluster.py 92.03% <63.63%> (-0.40%) ⬇️
redis/commands/json/__init__.py 97.56% <75.00%> (-2.44%) ⬇️
redis/commands/parser.py 88.40% <75.00%> (-1.92%) ⬇️
redis/commands/timeseries/__init__.py 94.73% <80.00%> (-5.27%) ⬇️
redis/commands/cluster.py 94.87% <100.00%> (+0.04%) ⬆️
tests/test_bloom.py 99.54% <100.00%> (+<0.01%) ⬆️
tests/test_graph.py 90.62% <100.00%> (+0.03%) ⬆️
tests/test_search.py 99.11% <100.00%> (+<0.01%) ⬆️
tests/test_timeseries.py 100.00% <100.00%> (ø)
redis/asyncio/client.py 90.45% <0.00%> (-0.18%) ⬇️
... and 1 more

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 c5d19b8...6fcfbaf. Read the comment docs.

@dvora-h dvora-h requested a review from chayim March 6, 2022 10:27
@chayim chayim merged commit 98fd06e into redis:master Mar 6, 2022
dvora-h added a commit to dvora-h/redis-py that referenced this pull request Mar 6, 2022
…edis#2032)

Co-authored-by: Chayim <chayim@users.noreply.github.com>
dvora-h added a commit that referenced this pull request Mar 6, 2022
* command docs

* Add support for AUTH (#1929)

* Add support for AUTH

* Fix linter error

* test fix

* fix test in cluster

Co-authored-by: Chayim <chayim@users.noreply.github.com>
Co-authored-by: Chayim I. Kirshen <c@kirshen.com>
Co-authored-by: dvora-h <dvora.heller@redis.com>

* Add support for JSON, TIMESERIES, BLOOM & GRAPH commands in cluster (#2032)

Co-authored-by: Chayim <chayim@users.noreply.github.com>

* Add support for HSET items (#2006)

* Add `items` parameter to `hset`

* Add test for `hset` with `items`

* Update CHANGES

* fix test_profile

Co-authored-by: Chayim <chayim@users.noreply.github.com>
Co-authored-by: dvora-h <dvora.heller@redis.com>

* Add cluster support for functions (#2016)

* cluster support for functions

* fix test_list_on_cluster mark

* fix mark

* cluster unstable url

* fix

* fix cluster url

* skip tests

* linters

* linters

* skip test

Co-authored-by: Chayim <chayim@users.noreply.github.com>
Co-authored-by: dogukanteber <47397379+dogukanteber@users.noreply.github.com>
Co-authored-by: Chayim I. Kirshen <c@kirshen.com>
Co-authored-by: Marek Czaplicki <mdczaplicki@gmail.com>
@dvora-h dvora-h deleted the json-cluster branch March 10, 2022 11:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants