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

Clean up connection handling and debugging #22

Closed
dzunk opened this issue Jul 18, 2020 · 0 comments · Fixed by #32
Closed

Clean up connection handling and debugging #22

dzunk opened this issue Jul 18, 2020 · 0 comments · Fixed by #32
Labels
chore Cleanup, refactoring, or some other hygiene task

Comments

@dzunk
Copy link
Owner

dzunk commented Jul 18, 2020

Kind of threw together the class-level connection handling code when adding .madd. This ought to be refactored to work with the instance-level client, and avoid copypasta'ing debug statements and redis.call everywhere.
@dzunk dzunk added the chore Cleanup, refactoring, or some other hygiene task label Jul 18, 2020
@dzunk dzunk mentioned this issue Jul 19, 2020
Merged
@dzunk dzunk closed this as completed in #32 Jul 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
chore Cleanup, refactoring, or some other hygiene task
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Extract client handling into Client module dzunk/redis-time-series
1 participant
@dzunk