Skip to content

Commit c75757f

Browse files
committed
fix types
1 parent baf44f8 commit c75757f

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

src/sentry/dynamic_sampling/tasks/boost_low_volume_projects.py

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -133,14 +133,19 @@ def partition_by_measure(
133133

134134
# Exclude orgs with project-mode sampling from the start. We know the
135135
# default is DynamicSamplingMode.ORGANIZATION.
136-
org_ids = {org.id for org, mode in modes.items() if mode != DynamicSamplingMode.PROJECT}
136+
filtered_org_ids = {
137+
org.id for org, mode in modes.items() if mode != DynamicSamplingMode.PROJECT
138+
}
137139

138140
if not options.get("dynamic-sampling.check_span_feature_flag"):
139-
metrics.incr("dynamic_sampling.partition_by_measure.transactions", amount=len(org_ids))
140-
return {SamplingMeasure.TRANSACTIONS: list(org_ids)}
141+
metrics.incr(
142+
"dynamic_sampling.partition_by_measure.transactions", amount=len(filtered_org_ids)
143+
)
144+
return {SamplingMeasure.TRANSACTIONS: list(filtered_org_ids)}
141145

142-
span_org_ids = set(options.get("dynamic-sampling.measure.spans")) or set()
143-
transactions_org_ids = org_ids - span_org_ids
146+
span_org_ids = set(options.get("dynamic-sampling.measure.spans") or [])
147+
span_org_ids = span_org_ids & filtered_org_ids
148+
transactions_org_ids = filtered_org_ids - span_org_ids
144149

145150
logger.info(
146151
"dynamic_sampling.partition_by_measure.options_check",

0 commit comments

Comments
 (0)