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.

Sign up for GitHub

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

Jump to bottom

feat(experimental): Basic implementation for the Redis Cluster client #235

Merged
merged 64 commits into from
Jul 11, 2021
Merged

feat(experimental): Basic implementation for the Redis Cluster client #235

merged 64 commits into from
Jul 11, 2021

Conversation

uki00a
Copy link
Member

@uki00a uki00a commented Jun 24, 2021 •
edited
Loading

Closes #220.

TODO

  • Re-enable temporarily disabled tests (2107ec6)
  • Add documents
  • tests, tests, tests

uki00a added 30 commits June 25, 2021 01:10
@uki00a
WIP: Basic implementation for Redis Cluster client
cdedf0e
@uki00a
chore: Fix typo
9f2132c
@uki00a
fix: Update
05baa00
@uki00a
test: Try to add tests for DEL command
2aebae7
@uki00a
chore: Ignore tests/server directory
d160c8a
@uki00a
chore: Fix
db2b2d5
@uki00a
chore: Fix comment
004070a
@uki00a
test: Try to setup Redis Cluster using redis-cli
7586b06
@uki00a
chore: Fixed lint
25386e8
@uki00a
ci: Allow Deno to run 'redis-cli'
29ec721
@uki00a
test: Failed to setup redis cluster...
ecaafbb
@uki00a
test: Improve error message
ae84469
@uki00a
test: Try to wait for servers to start up
30b95e2
@uki00a
fix: Update .gitignore
d23e2ec
@uki00a
test: Update cluster client tests
b22b839
@uki00a
chore: Debug
3c1125e
@uki00a
chore: Debug (part 2)
39f510a
@uki00a
chore: Lint
dbeb56f
@uki00a
test: Update cluster-config-file
8ead7c5
@uki00a
test: Update cluster-conf-file (part 2)
b4e1174
@uki00a
test: Remove debugging related stuff
0850c5e
@uki00a
test: Temporary disable all tests except those for redis cluster
2107ec6
@uki00a
chore: Lint
cfcd206
@uki00a
test: Add delay
c0353f1
@uki00a
test: Add "--cluster-yes" option
8fb8d4a
@uki00a
chore: Lint
37fbc26
@uki00a
test: Re-add delay
fac2172
@uki00a
fix: Fix cluster error handling
b33c357
@uki00a
chore: Debug
6ea1807
@uki00a
chore: Debug
9523561
uki00a added 22 commits July 10, 2021 15:30
@uki00a
chore: Rename key_test.ts to test.ts
fdba9b7
@uki00a
docs: Add README files
0654dd8
@uki00a
chore: Manage deps with dlink
a5a49b5
@uki00a
chore: Run "deno fmt"
59adefa
@uki00a
fix: Remove redisOptions and add newRedis instead
cf3abcf
@uki00a
test: Verifies a -MOVED error is correctly handled
03568fc
@uki00a
chore: Remove debugging things
d1c7d8e
@uki00a
test: Improve a test for a -MOVED error
0189a0c
@uki00a
fix: Update a test for a -MOVED error
91fdbba
@uki00a
fix: Mistakenly imported shuffle.js instead of sample.js...
45bdfdc
@uki00a
docs: Update experimental/cluster/README.md
c746bc4
@uki00a
chore: Remove duplicated code
76c9ce1
@uki00a
fix: Make maxConnections optional
3f4fa7e
@uki00a
chore: Add #getRedisLink method
d7928b5
@uki00a
test: Verify that a -ASK redirection error is correctly handled
e7aed82
@uki00a
fix: Try to correctly handle -ASK errors
fdfbb90
@uki00a
chore: Lint
e3f0726
@uki00a
docs: Improve docs
e7dbe95
@uki00a
chore: Move pasing logig to ClusterNode
84e6196
@uki00a
chore: Make config files in tests/tmp directory
3a0d5ad
@uki00a
Revert "test: Temporary disable all tests except those for redis clus…
526722a 
…ter"

This reverts commit 2107ec6.
@uki00a
chore: Run "deno fmt"
005b63e
@uki00a uki00a changed the title WIP: Implement Redis Cluster client feat(experimental): Basic implementation for the Redis Cluster client Jul 10, 2021
@uki00a uki00a marked this pull request as ready for review July 10, 2021 16:49
@uki00a uki00a merged commit de9576b into denodrivers:master Jul 11, 2021
@uki00a uki00a deleted the implement-redis-cluster-client branch July 11, 2021 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement ClusterExecutor
1 participant
@uki00a