-
Notifications
You must be signed in to change notification settings - Fork 340
Implement DBInterface/SonicV2Connector in C++ #387
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
eddb06e
[pyext] Add more OUTPUT type
qiluo-msft 639599d
Refactor: add new class RedisConnector
qiluo-msft 27dc476
dbconnector: remove emtpy line
qiluo-msft 9341111
Refine script functions parameter
qiluo-msft ee3a609
Add copy constructor to RedisConnector
qiluo-msft c358b24
Optimize DBConnector ctor
qiluo-msft eab9a9b
Revert back m_namespace
qiluo-msft 89bd455
Refactor: change name
qiluo-msft f11bca4
Add new ctor for DBConnector from RedisContext and dbId
qiluo-msft 5f4e3da
(ongoing) Add DBInterface class
qiluo-msft f731fcf
(temp, not build)
qiluo-msft c7b7023
Fix build
qiluo-msft 9b4d414
Extract psubscribe and subscribe function into DBConnector class
qiluo-msft d901fe5
Implement _subscribe_keyspace_notification, _unsubscribe_keyspace_not…
qiluo-msft 03b99c5
Implement blockable
qiluo-msft 1aa56b4
Implement connect with retry
qiluo-msft a8457a9
Implement DBConnector::publish(),
qiluo-msft 87bcf7f
Use c++11 syntax instead of c++14
qiluo-msft b0fbe03
Implement blocking for get and del
qiluo-msft bf779bd
Add to pyext
qiluo-msft c193a6a
Add set_redis_kwargs(), fix _onetime_connect()
qiluo-msft 9f0eb3e
Fix LGTM: delete implicitly-declared copy assignment operator
qiluo-msft e126eb1
update DBInterface redis_client index from db_id to db_name
qiluo-msft c754f15
Add DBInterface::delete_all_by_pattern()
qiluo-msft d282fbc
Add SonicV2Connector class
qiluo-msft c530fea
Add unit test for SonicV2Connector
qiluo-msft fdc256f
Make const strings public because they are used as public method default
qiluo-msft d4e442c
SWIG supports keyword arguments in generated python module
qiluo-msft 1d628ec
Add python namespace property to DBConnector class, solve the paramter
qiluo-msft 8c93402
Move SonicV2Connector to standalone .h/.cpp files
qiluo-msft f84f07b
Add missing include statements into SWIG inteface file
qiluo-msft 3a761e6
Add pytest unit test for DBInterface and SonicV2Connector
qiluo-msft 04545d7
Fix swig customization on SonicV2Connector ctor
qiluo-msft 2c8b889
Add attrib SonicV2Connector.namespace
qiluo-msft 11658b9
Remove debug code
qiluo-msft 832ace7
Use EXPECT_NE to simplify test
qiluo-msft b789a21
Remove unused code
qiluo-msft File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.