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

libmemcached: add MEMCACHED_BEHAVIOR_META_PROTOCOL #112

Draft
wants to merge 3 commits into
base: v1.x
Choose a base branch
from

Conversation

m6w6
Copy link
Collaborator

@m6w6 m6w6 commented Jun 9, 2021

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 15, 2021

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 70.92338% with 148 lines in your changes missing coverage. Please review.

Project coverage is 68%. Comparing base (bcc1a6e) to head (81b2016).
Report is 46 commits behind head on v1.x.

Files Patch % Lines
src/libmemcached/response.cc 65% 64 Missing ⚠️
src/libmemcached/get.cc 75% 20 Missing ⚠️
src/libmemcached/delete.cc 22% 18 Missing ⚠️
src/libmemcached/storage.cc 72% 13 Missing ⚠️
src/libmemcached/auto.cc 76% 9 Missing ⚠️
src/libmemcached/touch.cc 63% 7 Missing ⚠️
test/lib/ReturnMatcher.cpp 14% 6 Missing ⚠️
src/bin/common/options.hpp 40% 3 Missing ⚠️
src/libmemcached/behavior.cc 70% 3 Missing ⚠️
src/bin/memslap.cc 0% 1 Missing ⚠️
... and 4 more

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@          Coverage Diff           @@
##            v1.x    #112    +/-   ##
======================================
- Coverage     68%     68%    -0%     
======================================
  Files        194     194            
  Lines      17479   17812   +333     
======================================
+ Hits       11930   12150   +220     
- Misses      5549    5662   +113     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -96,14 +96,22 @@ memcached_return_t memcached_behavior_set(memcached_st *shell, const memcached_b
break;

case MEMCACHED_BEHAVIOR_BINARY_PROTOCOL:
send_quit(
ptr); // We need t shutdown all of the connections to make sure we do the correct protocol
// We need t shutdown all of the connections to make sure we do the correct protocol
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// We need t shutdown all of the connections to make sure we do the correct protocol
// We need to shutdown all of the connections to make sure we do the correct protocol

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.

3 participants