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

[MT-1660] - Batching and RemoteCommand processing fixes #248

Merged
merged 8 commits into from
Nov 21, 2024

Conversation

jameskeith
Copy link
Contributor

Pull request contains the following fixes

  • BugFix - ANR caused by slow processing of events upon backgrounding
  • BugFix - OOM error when dispatching larger queues; queues are now dispatched in batches, according to the configured batch size (min 1, max 10)
  • BugFix - Remote Command processing doesn't consider batching/connectivity/battery validators.
    • note. updates required to both core and remotecommanddispatcher package to avoid possible duplication of events

@jameskeith jameskeith requested a review from tamayok November 15, 2024 14:34
@jameskeith jameskeith merged commit f777025 into release/1.7.0 Nov 21, 2024
1 check passed
@jameskeith jameskeith deleted the mt-1660 branch November 21, 2024 13:53
jameskeith added a commit that referenced this pull request Nov 22, 2024
* [MT-1662] - Protect from uninitialized property error (#247)

* [MT-1664] - Support creating Tealium instances on non-main thread (#246)

* Init ContentProvider + ActivityManager

* Route activity events to listeners

* ActivityManager manual creation options, plus KDoc

* Tests + suppress errors

* keep TealiumInitProvider

* fix for failing test and doc updates

* Bump version to 1.7.0

* [MT-1660] - Batching and RemoteCommand processing fixes (#248)

* Batching revalidation on Tealium thread, batch size sanitization, Batching validator is now tolerant of missing first activity resume

* Batching sanitization

* on-backgrounding dispatch + remote command processing fixes

* Router and test fixes + RemoteCommandDispatcher fix and version update

* cleanup

* Post-merge import fix

* fix dispatchrouter imports

* Changelog updated
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