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

Document REDIS_NAMESPACE #5038

Merged
merged 1 commit into from Sep 22, 2017

Conversation

@nolanlawson
Copy link
Collaborator

commented Sep 22, 2017

I noticed that this feature (#2869) didn't seem to be documented anywhere. Seems .env.production.sample would be a good place for it?

@nolanlawson nolanlawson requested a review from beatrix-bitrot Sep 22, 2017
@clworld

This comment has been minimized.

Copy link
Collaborator

commented Sep 22, 2017

I wonder that is this feature works perfectly...
I believe that problems reported are fixed in nowadays..
but I sometimes heard that users trying use namespace / db with Redis got trouble...

@Gargron

This comment has been minimized.

Copy link
Member

commented Sep 22, 2017

@clworld the db option indeed does not work well, because redis pub/sub does not use db. However, namespace should be alright.

@Gargron Gargron merged commit d4f8082 into master Sep 22, 2017
3 checks passed
3 checks passed
codeclimate All good!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@nightpool

This comment has been minimized.

Copy link
Collaborator

commented Sep 22, 2017

@Gargron Gargron deleted the document-redis-namespace branch Sep 24, 2017
rutan added a commit to rutan/mastodon that referenced this pull request Oct 11, 2017
takayamaki added a commit to takayamaki/mastodon that referenced this pull request Oct 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.