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

Significant latency reduction in Redis #1361

Merged
merged 1 commit into from Jan 2, 2016

Conversation

Projects
None yet
2 participants
@etcimon
Contributor

etcimon commented Jan 1, 2016

This made my latency go down from ~400ms to ~100ms in the browser when requests required multiple cache subrequests.

Significant latency reduction in Redis
This made my latency go down from ~400ms to ~100ms in the browser when requests required multiple cache subrequests.
@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Jan 2, 2016

Member

Sounds fine. StreamOutputRange should make sure that data is sent in reasonable chunks, although we might want to extend it so that the buffer size can be configured.

Member

s-ludwig commented Jan 2, 2016

Sounds fine. StreamOutputRange should make sure that data is sent in reasonable chunks, although we might want to extend it so that the buffer size can be configured.

s-ludwig added a commit that referenced this pull request Jan 2, 2016

Merge pull request #1361 from etcimon/patch-32
Significant latency reduction in Redis

@s-ludwig s-ludwig merged commit 1df9746 into vibe-d:master Jan 2, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@etcimon etcimon deleted the etcimon:patch-32 branch Jan 2, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment