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

[ON HOLD] Support redis for duplicate cache #466

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

jfberry
Copy link
Collaborator

@jfberry jfberry commented Jun 21, 2021

Many users have redis already installed for MAD. This PR adds (optional) support to poracle to use this distributed cache as an alternative for the in-memory duplicate cache.
This will mean that events that have been processed will be remembered through restarts.

To activate, change the database, cache setting to redis. The other settings should be self explanatory.

      "cache": "redis",                // can be 'memory' or 'redis'
      "redis": {
        "host": "127.0.0.1",
        "port": 6379,
        "user": null,                   // default redis installation is no user/password but localhost only connections
        "password": null,
        "prefix": null,                 // usually leave as null! prefix all keys with this string (multi instance)
        "database": null                // usually leave as null! select this database on startup (advanced)
      }

Initial testing has not shown a significant improvement using this, so this requires further investigation. Please contact me before testing

@jfberry jfberry changed the title Support redis for duplicate cache [ON HOLD] Support redis for duplicate cache Jul 19, 2021
@jfberry jfberry added the invalid This doesn't seem right label Aug 20, 2021
# Conflicts:
#	config/default.json
#	src/app.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
experimental invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant