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

Redis for Session Storage? #42

Closed
nelsonic opened this issue Feb 19, 2015 · 5 comments
Closed

Redis for Session Storage? #42

nelsonic opened this issue Feb 19, 2015 · 5 comments
Labels
discuss Share your constructive thoughts on how to make progress with this issue help wanted If you can help make progress with this issue, please comment! investigate

Comments

@nelsonic
Copy link
Member

Should we use Redis for session storage or use ES for everything?
@nelsonic nelsonic added help wanted If you can help make progress with this issue, please comment! discuss Share your constructive thoughts on how to make progress with this issue investigate labels Feb 19, 2015
@nelsonic nelsonic mentioned this issue Feb 20, 2015
Closed
@nelsonic
Copy link
Member Author

Perhaps the best way to answer this is to do a performance test...?
I don't know if we are getting into _premature optimisation_ here...?
http://programmers.stackexchange.com/questions/80084/is-premature-optimization-really-the-root-of-all-evil

@nelsonic
Copy link
Member Author

Closing for now.
re-open if we have any latency issues with ES.
My highest read latency in esta on Travis-CI is 62ms ( at 81 records per second) which is terrible but more than fast enough for our MVP.
see: https://travis-ci.org/nelsonic/esta/builds/49519538#L246

@nelsonic nelsonic mentioned this issue Apr 16, 2015
Closed
@nelsonic
Copy link
Member Author

Redis vagrant: http://stackoverflow.com/questions/27151556/what-is-the-simplest-way-to-install-redis-in-dockervagrant

@nelsonic nelsonic reopened this Apr 21, 2015
nelsonic added a commit that referenced this issue Apr 21, 2015
@nelsonic
adds redis connection helper & test #42
6d6411a
@nelsonic nelsonic mentioned this issue Apr 21, 2015
Merged
nelsonic added a commit that referenced this issue Apr 22, 2015
@nelsonic
[WiP] adds *LOTS* of debug console.logs #42
0c286a6
@nelsonic nelsonic mentioned this issue Apr 23, 2015
Closed
nelsonic added a commit that referenced this issue Apr 23, 2015
@nelsonic
logout tests working with redis. getting there. #42
69db0c4
@nelsonic
Copy link
Member Author

nelsonic commented Apr 27, 2015 •
edited
Loading

@iteles iteles added [alpha] and removed Post MVP labels Apr 27, 2015
@nelsonic nelsonic mentioned this issue Jul 12, 2015
Closed
@nelsonic
Copy link
Member Author

We will be using PostgreSQL for Session Storage and ETS for session invalidation. see: dwyl/auth#30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
discuss Share your constructive thoughts on how to make progress with this issue help wanted If you can help make progress with this issue, please comment! investigate
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nelsonic @iteles