Skip to content

Commit 81ddfb0

Browse files
untitakerandrewshie-sentry
authored andcommitted
ref(span-buffer): Move max-memory-percentage to right CLI (#91924)
This was added to the wrong CLI, and because it's an optional argument in the factory this wasn't noticed during manual testing.
1 parent ac133fe commit 81ddfb0

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/sentry/consumers/__init__.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -424,6 +424,11 @@ def ingest_transactions_options() -> list[click.Option]:
424424
default=100,
425425
help="The number of segments to download from redis at once. Defaults to 100.",
426426
),
427+
click.Option(
428+
["--max-memory-percentage", "max_memory_percentage"],
429+
default=1.0,
430+
help="Maximum memory usage of the Redis cluster in % (0.0-1.0) before the consumer backpressures.",
431+
),
427432
*multiprocessing_options(default_max_batch_size=100),
428433
],
429434
},
@@ -436,11 +441,6 @@ def ingest_transactions_options() -> list[click.Option]:
436441
is_flag=True,
437442
default=False,
438443
),
439-
click.Option(
440-
["--max-memory-percentage", "max_memory_percentage"],
441-
default=1.0,
442-
help="Maximum memory usage of the Redis cluster in % (0.0-1.0) before the consumer backpressures.",
443-
),
444444
*multiprocessing_options(default_max_batch_size=100),
445445
],
446446
},

0 commit comments

Comments
 (0)