- User Since
- Jul 8 2019, 5:19 PM (39 w, 1 d)
Hm the speedup isn't that significant when we run all the tests, but it does seem like at least 1 - 2 minute speedup. Doesn't feel like it's worth it. Will play around with this more
tweaking some numbers
Reset cursor if stepping up to a bigger page size
turn down parallelism
tox env syntax
Mon, Apr 6
Sun, Apr 5
Remove unecessary fragment
Abandoning in favor of https://dagster.phacility.com/D2446
Implemented in https://dagster.phacility.com/D2430
Make sure you add get_run_groups to the RunStorage base class.
Great! Let's get this landed 🙌
Actually, it was my mistake in writing the test. Run order is good 👍
Sat, Apr 4
Fix line numbers
Fri, Apr 3
Awesome work! Super impressed by the attention to detail. I love what you did with the literal include.
Thu, Apr 2
A quick hack might be to expose a validation method on the RunLauncher, and only call create_run on the instance if launcher.validate() returns True.
You probably already considered this, but just to flag: this would change the behavior of RemoteDagitRunLauncher.
Awesome stuff :)
Wed, Apr 1
Tue, Mar 31
Add back string replacement
Looks good! Definitely enough here to start building the view