8000 feat: Added ODPManager implementation by mikechu-optimizely · Pull Request #322 · optimizely/csharp-sdk · GitHub
[go: up one dir, main page]
Skip to content

feat: Added ODPManager implementation #322

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.

Sign up for GitHub

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 18 commits into from
Dec 7, 2022
Merged

feat: Added ODPManager implementation #322

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
dd028bf
Initial ODP Manager
mikechu-optimizely Nov 23, 2022
a317407
Outline unit tests
mikechu-optimizely Nov 23, 2022
443164c
WIP Unit tests + changes to CUD
mikechu-optimizely Nov 28, 2022
faea784
Implement fetch, identify, and send events with Builder pattern
mikechu-optimizely Nov 28, 2022
191a6ec
WIP Fixing final unit test
mikechu-optimizely Nov 28, 2022
f6276bc
Finish unit tests
mikechu-optimizely Nov 29, 2022
4f629fa
Add documentation
mikechu-optimizely Nov 29, 2022
bbf8ac1
Lint fixes
mikechu-optimizely Nov 29, 2022
4a54b4c
Pull request code revisions + additional unit tests
mikechu-optimizely Dec 1, 2022
0995694
Lint fixes
mikechu-optimizely Dec 1, 2022
88b6684
Change to use string array over List
mikechu-optimizely Dec 1, 2022
29514ff
Clean usings
mikechu-optimizely Dec 1, 2022
22b04ae
Add new OdpConfig test
mikechu-optimizely Dec 1, 2022
dcafcee
Small refactors
mikechu-optimizely Dec 1, 2022
8747556
Pull request code review changes
mikechu-optimizely Dec 2, 2022
dfaa27b
Code review changes
mikechu-optimizely Dec 6, 2022
c528dce
Mark _odpConfig as volatile
mikechu-optimizely Dec 6, 2022
a5b28e6
Return `volatile` denotation
mikechu-optimizely Dec 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Mark _odpConfig as volatile
  • Loading branch information
@mikechu-optimizely
mikechu-optimizely committed Dec 6, 2022
commit c528dce762c757ba6ef37249fe81c14ca26e6232
2 changes: 1 addition & 1 deletion OptimizelySDK/Odp/OdpEventManager.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ namespace OptimizelySDK.Odp
/// </summary>
public class OdpEventManager : IOdpEventManager, IDisposable
{
private OdpConfig _odpConfig;
private volatile OdpConfig _odpConfig;
private IOdpEventApiManager _odpEventApiManager;
private int _batchSize;
private TimeSpan _flushInterval;
Expand Down
2 changes: 1 addition & 1 deletion OptimizelySDK/Odp/OdpSegmentManager.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ public class OdpSegmentManager : IOdpSegmentManager
/// <summary>
/// ODP configuration containing the connection parameters
/// </summary>
private OdpConfig _odpConfig;
private volatile OdpConfig _odpConfig;

/// <summary>
/// Cached segments
Expand Down
0