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

Bug 1918144 - Document submitting Glean events in "real-time" #2953

Merged
merged 2 commits into from
Sep 12, 2024

Conversation

travis79
Copy link
Member

This adds some more specific documentation about Glean's ability to submit events in "real-time" (sending, not analyzing) by way of using the maximum events per ping parameter to cause Glean to send an events ping for every event recorded.

This will be augmented later once I've gotten a chance to run some experiments and measure the impacts with real data.

@travis79 travis79 requested a review from a team as a code owner September 11, 2024 18:37
@travis79 travis79 requested review from chutten and removed request for a team September 11, 2024 18:37
Copy link
Contributor

@chutten chutten left a comment

Choose a reason for hiding this comment

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

The doc's a little slim on reasons why someone might choose to configure a Glean SDK in this way. Is this intentional, or should we write a sentence like "this, in ideal network and ingestion circumstances, can allow data to arrive within seconds-to-minutes of being recorded"

@travis79
Copy link
Member Author

The doc's a little slim on reasons why someone might choose to configure a Glean SDK in this way. Is this intentional, or should we write a sentence like "this, in ideal network and ingestion circumstances, can allow data to arrive within seconds-to-minutes of being recorded"

It's light on marketing details because I still have some work and measurement of impacts that I'd like to do to complete this documentation. I'd like to be able to back up any claims and potential impacts with real data. This'll be tracked in Bug 1918149

@travis79 travis79 merged commit c33b53c into main Sep 12, 2024
29 of 30 checks passed
@travis79 travis79 deleted the Bug1918144 branch September 12, 2024 14:28
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