Skip to content

Commit

Permalink
Add buffer time to perf test (#25915)
Browse files Browse the repository at this point in the history
## Summary & Motivation

As title. This is a consequence of getting rid of the special-purpose
PartitionKeysTimeWindowPartitionsSubset. Long story short, these perf
tests are not very representative of real-world perf scenarios, and
mostly exist as a backstop in case anything massively changes. This
change is also completely irrelevant to the Declarative Automation perf
characteristics, which are more important and have their own separate
perf tests.

TL;DR: We'll probably remove this test in the near future, for now we're
just making it stop failing

## How I Tested These Changes

## Changelog

NOCHANGELOG
  • Loading branch information
OwenKephart authored Nov 13, 2024
1 parent dcdf600 commit 691d93e
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@
),
all_daily_partitioned_500_assets.build_scenario(
partition_keys_to_backfill=[f"2020-01-{i+1:02}" for i in range(20)],
max_execution_time_seconds=30,
max_execution_time_seconds=40,
),
all_hourly_partitioned_100_assets.build_scenario(
partition_keys_to_backfill=["2020-01-01-00:00", "2020-01-02-00:00"],
Expand Down

0 comments on commit 691d93e

Please sign in to comment.