From 755e9e3ca9a33c53be393b4613fa86aa95b0234c Mon Sep 17 00:00:00 2001 From: William Mak Date: Mon, 24 Nov 2025 15:45:19 -0500 Subject: [PATCH] feat(eap): Always log the rpc query instead of only at debug - Gonna just try this out, we'll keep an eye on how much we're sending and tune it accordingly --- src/sentry/snuba/rpc_dataset_common.py | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/src/sentry/snuba/rpc_dataset_common.py b/src/sentry/snuba/rpc_dataset_common.py index 28d528f5b3d21d..e84d7dcbf78525 100644 --- a/src/sentry/snuba/rpc_dataset_common.py +++ b/src/sentry/snuba/rpc_dataset_common.py @@ -334,8 +334,7 @@ def _run_table_query( """Run the query""" table_request = cls.get_table_rpc_request(query) rpc_request = table_request.rpc_request - if debug: - log_rpc_request("Running a table query with debug on", rpc_request) + log_rpc_request("Running a table query with debug on", rpc_request) try: rpc_response = snuba_rpc.table_rpc([rpc_request])[0] except Exception as e: @@ -545,8 +544,7 @@ def update_timestamps( def _run_timeseries_rpc( self, debug: bool, rpc_request: TimeSeriesRequest ) -> TimeSeriesResponse: - if debug: - log_rpc_request("Running a timeseries query with debug on", rpc_request) + log_rpc_request("Running a timeseries query with debug on", rpc_request) try: return snuba_rpc.timeseries_rpc([rpc_request])[0] except Exception as e: @@ -809,9 +807,8 @@ def run_top_events_timeseries_query( requests.append(other_request) """Run the query""" - if params.debug: - for rpc_request in requests: - log_rpc_request("Running a top events query with debug on", rpc_request) + for rpc_request in requests: + log_rpc_request("Running a top events query with debug on", rpc_request) try: timeseries_rpc_response = snuba_rpc.timeseries_rpc(requests) rpc_response = timeseries_rpc_response[0]