Skip to content

Added support for partial consumer mode - #61

Merged
EmilianoSanchez merged 7 commits into
consumer_mode_client_sidefrom
consumer_mode_partial
Nov 24, 2021
Merged

Added support for partial consumer mode#61
EmilianoSanchez merged 7 commits into
consumer_mode_client_sidefrom
consumer_mode_partial

Conversation

@EmilianoSanchez

@EmilianoSanchez EmilianoSanchez commented Nov 11, 2021

Copy link
Copy Markdown
Contributor

Javascript commons library

What did you accomplish?

Added support for partial consumer mode:

  • Updated PluggableStorage to work with in memory versions of events and impressions cache, and support optimized mode for imp. dedup.
  • Updated SyncManager to be instantiated with submitters but not polling manager.
  • Updated mode validator, to address the new mode type 'consumer_partial'.
  • Updated storage validator, to throw error on invalid combinations of mode and storage type (e.g., consumer mode requires custom/pluggable storage).

How do we test the changes introduced in this PR?

  • Updated tests

Extra Notes

TODO: review mode & storage defaults and validation

@EmilianoSanchez
EmilianoSanchez merged commit 5c675d2 into consumer_mode_client_side Nov 24, 2021
@EmilianoSanchez
EmilianoSanchez deleted the consumer_mode_partial branch November 24, 2021 17:36
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