Skip to content

fix: flush interval flushes only if the threshold is met#7435

Closed
marandaneto wants to merge 1 commit intomasterfrom
fix/flush-android
Closed

fix: flush interval flushes only if the threshold is met#7435
marandaneto wants to merge 1 commit intomasterfrom
fix/flush-android

Conversation

@marandaneto
Copy link
Member

Changes

https://posthog.com/questions/flush-interval-seconds-not-working
Also PostHog/posthog-android#73

the flush interval only flushes events only if the threshold (flushAt) is met to avoid high battery consumption.
if the flush interval flushes events regardless of the total of events, every 30s, most likely the batch operation won't be that useful since you don't capture many events every 30s, so it's better to wait for the threshold (flushAt) instead.
The alternative is to call flush, in this case, events are sent right away, the fix for that is here.

Checklist

  • Words are spelled using American English
  • Titles are in sentence case
  • Feature names are in sentence case too
  • Use relative URLs for internal links
  • If I moved a page, I added a redirect in vercel.json
  • Remove this template if you're not going to fill it out!

Article checklist

  • I've added (at least) 3-5 internal links to this new article
  • I've added keywords for this page to the rank tracker in Ahrefs
  • I've checked the preview build of the article
  • The date on the article is today's date
  • I've added this to the relevant "Tutorials and guides" docs page (if applicable)

Useful resources

@vercel
Copy link

vercel bot commented Jan 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
posthog ✅ Ready (Inspect) Visit Preview Jan 3, 2024 11:27am

Copy link
Member

@pauldambra pauldambra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@marandaneto
Copy link
Member Author

Closed in favor of PostHog/posthog-android#73
the SDK meets the description in the current form

@marandaneto marandaneto closed this Jan 3, 2024
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