* docs(community): Add Code of Conduct #107
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* feat(pre-commit): Add Conventional Commits `commit-msg` hook #107
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* bug(pre-commit): Fix conventional commits entry to include `contrib/scripts` #107
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* feat(docker-compose): Add `docker-compose.yml` #100
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* docs(build-from-source): Include build from source
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* Reduce scope of HELLO implementation
Only accept protover=2 as a valid argument as that is
the only thing that is supported at the moment. For all
other arguments degrade to 'unknown command'
The previous implementation creates issues for clients
expecting the presence of the HELLO command to also signal
the presence of RESP3 (as technically the command appeared in
redis at the same time as support for RESP3).
It also did not raise any errors when unsupported (or invalid)
arguments were passed to the command (such as AUTH, SETNAME)
* Fix error in test
* Remove unnecessary context in assertion
* Fix construction of unknown command error string
* Fix casing of error string
* Add tests demonstrating expectations of (p)unsubscribe
- When there are subscriptions the return should contain the count of
remaining channels or patterns subscribed to
- When there are no subscriptions and no arguments are provided
a single response containing null for channel / pattern and count = 0
should be returned
- When there are subscriptions and no arguments are provided
a response per unsubscribed channel or pattern should be returned
* Update arity of (p)unsubscribe commands
* Lint test
* Handle (p)unsubscribe without arguments
Side effects:
- Extract response construction for (p)sububscribe, (p)unsubscribe
into private method
- Use UnsubscribeAll, PunsubscribeAll in cleanup
* Fix linting errors
* Replace ternary with explicit if/else
* Simplify setup for (p)unsubscribe tests
* Update response to HELLO command
Update response from HELLO command to match the api of redis.
Notes:
- For any protocol version other than 2 an error matching what redis
returns for !2,3 is returned
* Add test for redis HELLO command
* Extract version building to GetVersion function
* Update contributors file
* Add HELLO command to README readiness matrix
* Revert "Add HELLO command to README readiness matrix"
This reverts commit 069f590ad0.
* Add HELLO command to api_status document
* Fix `SubscribeMap`
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* Fix `subscriber_arr`
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* Fix `CopySubscribers`
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* Fix `SubscriberMessagesLen`
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* Fix remaining `subscribe` variants
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* Add Ryan Russell to Contributors
Signed-off-by: Ryan Russell <git@ryanrussell.org>
* Initial commit for helm chart
Includes support for
- persistence for /data (if enabled, a Statefulset is deployed)
- mounting extra volumes (for logs, debugging, whatever)
- initContainers (adjusting the pod's ulimit, possibly)
- passing extra arguments to the dragonfly binary through extraArgs
Squashed commit of the following:
caa91a0 helm-chart: initial commit
7ec9ea5 helm-chart: add extraArgs and update README/TODO
e1da96c helm-chart: add StatefulSet for persistence and update chart README
4d81f8a helm-chart: add liveness+readinessProbe
cdf70b3 helm-chart: add initContainers, extraVolumes and extraVolumeMounts
aed0ef1 helm-chart: update README
* add CONTRIBUTORS
Ryan Russell
Ali-Akber Saifee
Roman Gershman
Roman Gershman
Philipp B
odedponcz
Zacharya
TAKAHASHI Shuuji
Philipp B
quiver
Olle Jonsson
James Herdman