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

redis 2.0 #56

Merged
merged 8 commits into from
Mar 18, 2017
Merged

redis 2.0 #56

merged 8 commits into from
Mar 18, 2017

Conversation

tanner0101
Copy link
Member

@tanner0101 tanner0101 commented Mar 16, 2017

  • 5x faster
  • 3.5k lines of code removed

2.0

screen shot 2017-03-16 at 7 22 39 pm

1.0

screen shot 2017-03-16 at 7 22 47 pm

Todo:

  • Transport Streams from Engine

@tanner0101 tanner0101 added the enhancement New feature or request label Mar 16, 2017
@tanner0101 tanner0101 added this to the 2.0 milestone Mar 16, 2017
@tanner0101 tanner0101 self-assigned this Mar 16, 2017
@jdmcd
Copy link
Member

jdmcd commented Mar 16, 2017

Would it be possible to implement an expiration time on Redis using EXPIRE?

@tanner0101
Copy link
Member Author

tanner0101 commented Mar 16, 2017

@mcdappdev besides supporting expire in the Command enum, idk how much use that would be here. But adding that to the CacheProtocol in vapor would be awesome!

@jdmcd
Copy link
Member

jdmcd commented Mar 16, 2017 via email

@tanner0101
Copy link
Member Author

@mcdappdev sure thing :) I should have this merged and tagged once I pull in the proper Stream protocols (will be done in 12-24 hours)

here's the cache protocol to start with
https://github.com/vapor/vapor/blob/9b0701479b39996b56ada92565cded106c73b57b/Sources/Cache/CacheProtocol.swift

@jdmcd
Copy link
Member

jdmcd commented Mar 16, 2017 via email

@vzsg
Copy link
Member

vzsg commented Mar 17, 2017

@mcdappdev I wrote a proposal with CacheProtocol changes after your idea, back when we were discussing this on Slack 😛 Apparently it flew below the radar: vapor/vapor#907

@jdmcd
Copy link
Member

jdmcd commented Mar 17, 2017

@vzsg I love it! I just left one comment but other than that I'm a big fan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants