Skip to content

Commit

Permalink
fix: Only use major OS version in Apple symbol stats (#78765)
Browse files Browse the repository at this point in the history
Using the full OS version has a very high cardinality, so we truncate at
the first `.` to get just the major version.
  • Loading branch information
loewenheim authored Oct 8, 2024
1 parent f273874 commit 0213755
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/sentry/lang/native/processing.py
Original file line number Diff line number Diff line change
Expand Up @@ -486,6 +486,9 @@ def emit_apple_symbol_stats(apple_symbol_stats, data):
)
os_version = get_path(data, "contexts", "os", "version")

if os_version:
os_version = os_version.split(".", 1)[0]

if neither := apple_symbol_stats.get("neither"):
metrics.incr(
"apple_symbol_availability_v2",
Expand Down

0 comments on commit 0213755

Please sign in to comment.