From d376ed6a9b023d6abc81cb4bd83fc86d3dee527d Mon Sep 17 00:00:00 2001 From: Andrew Liu <159852527+aliu39@users.noreply.github.com> Date: Mon, 15 Jul 2024 08:54:30 -0700 Subject: [PATCH 1/2] metrics(replay): log rrweb_event_count to slow clicks table --- src/sentry/replays/usecases/ingest/__init__.py | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/src/sentry/replays/usecases/ingest/__init__.py b/src/sentry/replays/usecases/ingest/__init__.py index c21d3af01ac565..9a60fcb185f9c2 100644 --- a/src/sentry/replays/usecases/ingest/__init__.py +++ b/src/sentry/replays/usecases/ingest/__init__.py @@ -261,6 +261,23 @@ def recording_post_processor( message.replay_id, parsed_segment_data, ) + + # Log # of rrweb events to bigquery. + logger.info( + # Logging to the sentry.replays.slow_click namespace because + # its the only one configured to use BigQuery at the moment. + # + # NOTE: Needs an ops request if we want to create a new dataset. + "sentry.replays.slow_click", + extra={ + "event_type": "rrweb_event_count", + "org_id": message.org_id, + "project_id": message.project_id, + "replay_id": message.replay_id, + "size": len(parsed_segment_data), + }, + ) + except Exception: logging.exception( "Failed to parse recording org=%s, project=%s, replay=%s, segment=%s", From 13831d37a4c63dc4ecbd404a7130977c005bfcd4 Mon Sep 17 00:00:00 2001 From: Andrew Liu <159852527+aliu39@users.noreply.github.com> Date: Mon, 15 Jul 2024 08:58:52 -0700 Subject: [PATCH 2/2] Rm comment --- src/sentry/replays/usecases/ingest/__init__.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/sentry/replays/usecases/ingest/__init__.py b/src/sentry/replays/usecases/ingest/__init__.py index 9a60fcb185f9c2..17f0fcf9d3ece7 100644 --- a/src/sentry/replays/usecases/ingest/__init__.py +++ b/src/sentry/replays/usecases/ingest/__init__.py @@ -264,10 +264,6 @@ def recording_post_processor( # Log # of rrweb events to bigquery. logger.info( - # Logging to the sentry.replays.slow_click namespace because - # its the only one configured to use BigQuery at the moment. - # - # NOTE: Needs an ops request if we want to create a new dataset. "sentry.replays.slow_click", extra={ "event_type": "rrweb_event_count",