1798713 - Increase Glean Events limits (properties, size)

Status: RESOLVED FIXED

(Data Platform and Tools :: Glean: SDK, enhancement, P1)

Description

[Marco Bonardo [:mak]]

For the new engagement telemetry in the urlbar, we need larger limits than the current one.
We currently have 11 properties, and the longest property is a CSV string made of result types. The longest result type we have is 20 bytes, most users will have 10 results, but some may increase that value. Though, we expect results over a certain threshold (let's say 25 results) won't be particularly usable.
Thus, I think meaningful sizes, also to keep into account potential future needs, would be:

• 15 properties
• 500 bytes value length. ideally the worst case scenario would be 25 results * 20 chars + 24 commas, though most users have 10 results and most types are shorter, thus we think most cases will be under 200 bytes, and this is a good enough worst case scenario.

Comment 1

[Jan-Erik Rediger [:janerik]]

Attachment: pr 525: Bug 1798713 - events: Increase extras limit to 15

Comment 2

[Jan-Erik Rediger [:janerik]]

badboy merged PR #525: "Bug 1798713 - events: Increase extras limit to 15" in e60a0f5.

Comment 3

[Jan-Erik Rediger [:janerik]]

Attachment: pr 2255: Bug 1798713 - Events: Increase event extras limit to 15 and event extra value limit to 500 bytes

Comment 4

[Jan-Erik Rediger [:janerik]]

badboy merged PR #2255: "Bug 1798713 - Events: Increase event extras limit to 15 and event extra value limit to 500 bytes" in 298108e.

This is going into a release today.

Comment 5

[Jan-Erik Rediger [:janerik]]

This is in v51.8.0.
bug 1799129 is for landing this in m-c and subsequently in Fenix.