-
Notifications
You must be signed in to change notification settings - Fork 3
feat: Implement polling data source. #177
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
Closed
kinyoklion
wants to merge
65
commits into
server-side
from
rlamb/implement-server-polling-data-source
Closed
Changes from all commits
Commits
Show all changes
65 commits
Select commit
Hold shift + click to select a range
95bab24
chore: Implement architecture diagram for data store.
kinyoklion a4b2c8c
Updates types for get methods on IDataStore.
kinyoklion d368988
Update collection types.
kinyoklion 48bfc55
Merge branch 'server-side' into rlamb/data-store-arch
kinyoklion 758e825
feat: Implement basic store.
kinyoklion 6914703
Overloads for upsert and add extra const qualifiers.
kinyoklion c74f508
Merge branch 'rlamb/data-store-arch' of github.com:launchdarkly/cpp-s…
kinyoklion cd1ec6f
Merge branch 'rlamb/data-store-arch' into rlamb/basic-store-implement…
kinyoklion e82dc04
Basic data store and data source update sink interfaces.
kinyoklion 54499d6
Start adding change notifier.
kinyoklion 1df05f9
Merge branch 'server-side' into rlamb/basic-store-implementation
kinyoklion 5221784
Add empty implementation file.
kinyoklion e9734bc
Start adding memory store
kinyoklion b53b004
Basic memory store functionaliy.
kinyoklion 1d07b00
Add memory store tests.
kinyoklion 58d284f
Add tests for dependency tracker.
kinyoklion f6b9558
Split implementation and cleanup clang issues.
kinyoklion d4a2a9d
Refactoring and continue implementation of the data store updater.
kinyoklion 091bcb8
Code cleanup and more tests.
kinyoklion f3e0a5d
Add a test for using an old value after upsert.
kinyoklion 0bd9846
Merge branch 'server-side' into rlamb/basic-store-implementation
kinyoklion 2fcca51
Cleanup cmakelists.txt.
kinyoklion eedb155
Merge branch 'rlamb/basic-store-implementation' of github.com:launchd…
kinyoklion f2f4fe6
Add explainer comment for event handler.
kinyoklion 2fb9abb
Linting
kinyoklion b33ae7f
Handle added and deleted. Add new test for update, add, delete, and u…
kinyoklion 1e37fe3
Move IDataSource and make data_sources a consistent directory and nam…
kinyoklion 7667740
Move data source status.
kinyoklion e2d205a
Reformatting.
kinyoklion 0561288
Const
kinyoklion e530857
Update libs/server-sdk/src/data_store/data_store.hpp
kinyoklion e62258b
Merge branch 'rlamb/basic-store-implementation' of github.com:launchd…
kinyoklion 057355e
Aliases and descriptors
kinyoklion a3614f7
More PR feedback updates.
kinyoklion e3fadc0
Fallthrough
kinyoklion b8fb098
Remove comment.
kinyoklion fe27583
More const and lint
kinyoklion 7eb0023
Tidy
kinyoklion 5cef5f3
More kind usage.
kinyoklion 6618806
Fix namespace comment.
kinyoklion a0a62b2
Merge branch 'rlamb/basic-store-implementation' into rlamb/implement-…
kinyoklion 1d309f5
Enable shared data source status.
kinyoklion 1155970
Add missing newlines.
kinyoklion d83ea5a
Merge branch 'server-side' into rlamb/refactor-common-store-code
kinyoklion 2c547ca
Lint
kinyoklion 278487c
Attempt to appease lint.
kinyoklion fff7dbf
More cleanup
kinyoklion b274a35
More linting
kinyoklion 6b272f0
Remove CRTP
kinyoklion 367288c
Remove CRTP from tests.
kinyoklion d4f980f
Add missing cstdint header.
kinyoklion e91be14
Delete redundant folder.
kinyoklion 36766ce
feat: Implement polling data source.
kinyoklion 0fd89d9
Implement basic status handler and event handler.
kinyoklion 8dc5567
Handle polling payload.
kinyoklion f2269bb
Implement polling.
kinyoklion fa5c96b
Merge branch 'server-side' into rlamb/implement-server-polling-data-s…
kinyoklion 2178df6
Basic polling works.
kinyoklion ff4e5bf
cleanup
kinyoklion 4d927d5
Revert unintended status change.
kinyoklion d0ee2ed
Correct status for server.
kinyoklion 2698adc
Remove redundant base initializer.
kinyoklion e1266fc
Reformat data_source_status_manager.
kinyoklion cab59ea
Lint and cleanup.
kinyoklion 03b9b27
Revert polling fix for alternate PR.
kinyoklion 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
129 changes: 0 additions & 129 deletions
129
libs/client-sdk/src/data_sources/data_source_status_manager.cpp
This file was deleted.
Oops, something went wrong.
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
Oops, something went wrong.
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.