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

Limit Queue size to 1000 #565

Merged
merged 1 commit into from
Jun 27, 2016
Merged

Limit Queue size to 1000 #565

merged 1 commit into from
Jun 27, 2016

Conversation

f2prateek
Copy link
Contributor

@f2prateek f2prateek commented Jun 27, 2016

This limits the queue to 1000 events (like we do on Android). This caps the size to 1000 * 15kb = 15mb. We can improve this in the future by moving to Cassette so that we don't need to hold events in memory and writes run in O(1) rather than O(n) https://github.com/segmentio/cassette.

@f2prateek f2prateek merged commit e18ef89 into dev Jun 27, 2016
@f2prateek f2prateek deleted the limit branch June 27, 2016 14:37
f2prateek added a commit that referenced this pull request Jun 30, 2016
f2prateek added a commit that referenced this pull request Jul 10, 2016
f2prateek added a commit that referenced this pull request Jul 10, 2016
* Carthage Support

* Initialize flush timer on the main thread.

* Replace Foundation import with UIKit import (#563)

* Track Campaign Data (#557)

* Update contributing docs (#564)

* Limit Queue size to 1000 (#565)

* Reformat (#566)

* Exclude files from backup (#567)

* Add tvOS Support (#572)

* reformat

* Track Deep Links (#573)

* Fix CI (#574)

* Use vanilla xcodebuild in CI

* use xcpretty in ci

* Update context object with referrer information (#575)

* use 3.2.6 in sample
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.

1 participant