- User Since
- Jul 6 2020, 12:49 PM (41 w, 2 h)
Sat, Apr 17
Fri, Apr 16
I'd like to try to cherry pick this if reviewers have a sec (since the blamerev is going out in the release)
Thu, Apr 15
sorry for all the back and forth here, just looking to get the copy super clear since the early check is a common source of user confusion in #general
Wed, Apr 14
Tue, Apr 13
this generally makes a ton of sense to me, I just had some more time to think about the exact copy to use for maximum clarify, left some thoughts inline.
@prha may have thoughts here too.
Mon, Apr 12
Sat, Apr 10
Fri, Apr 9
This might be better suited for the specific subclass you’re working with that’s already associated with an instance?
Thu, Apr 8
should some test have broken that this would fix?
Wed, Apr 7
ExecutorProcessSetting => List[ExecutorRequirement]
@alex you have accepted the two follow-on diffs to this but the base of the stack remains :) ExecutorProcessSetting giving you pause?
flake test failure
no need for a single metadata
Tue, Apr 6
use env vars
sorry for sitting on this forever.
enum on the ExecutorDefinition
Mon, Apr 5
probably 80% of this diff is passing DagsterInstances around so that we can access it during plan creation to validate the executor, I could split that part out into a separate diff if that would make it easier to review