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

[WIP] Argument caching #442

Merged
merged 6 commits into from
Apr 18, 2016
Merged

[WIP] Argument caching #442

merged 6 commits into from
Apr 18, 2016

Conversation

kmike
Copy link
Member

@kmike kmike commented Apr 14, 2016

This PR should allow to save network traffic by caching large argument values on server.

See also: #283.

@codecov-io
Copy link

Current coverage is 85.85%

Merging #442 into master will increase coverage by +0.14% as of 709ca8a

@@            master    #442   diff @@
======================================
  Files           39      40     +1
  Stmts         4628    4729   +101
  Branches       633     652    +19
  Methods          0       0       
======================================
+ Hit           3967    4060    +93
- Partial        156     159     +3
- Missed         505     510     +5

Review entire Coverage Diff as of 709ca8a

Powered by Codecov. Updated on successful CI builds.

@kmike kmike merged commit 6f40a0a into master Apr 18, 2016
@kmike kmike deleted the cache-args branch May 16, 2016 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants