User Details
User Details
- User Since
- Jul 6 2020, 12:49 PM (28 w, 6 d)
Yesterday
Yesterday
squashed into https://dagster.phacility.com/D6133
squashed into https://dagster.phacility.com/D6133
squashed into https://dagster.phacility.com/D6133
dgibson abandoned D6086: [5/n] Wait to determine step compute_fn until you're actually in step execution.
squashed into https://dagster.phacility.com/D6133
squashed into https://dagster.phacility.com/D6133
dgibson retitled D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists from (Squashed) Reconstruct ExecutionPlan from the ExecutionPlanSnapshot to Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.
up
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.
up
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.
up
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.
up
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.
up
dgibson updated the diff for D6133: Reconstruct ExecutionPlan from the ExecutionPlanSnapshot in the run worker if one exists.
up
Sat, Jan 23
Sat, Jan 23
dgibson added a comment to D6131: Replace dagster api grpc with 'python -m dagster.grpc' in docs?.
"As initialized upon program startup, the first item of this list, path[0], is the directory containing the script that was used to invoke the Python interpreter. If the script directory is not available (e.g. if the interpreter is invoked interactively or if the script is read from standard input), path[0] is the empty string, which directs Python to search modules in the current directory first."
dgibson added a comment to D6131: Replace dagster api grpc with 'python -m dagster.grpc' in docs?.
possibly relevant:
dgibson committed R1:ac7dec89ecca: Remove pipeline_def from top-level SystemExecutionContext, only use in step… (authored by dgibson).
Remove pipeline_def from top-level SystemExecutionContext, only use in step…
dgibson published D6131: Replace dagster api grpc with 'python -m dagster.grpc' in docs? for review.
dgibson updated the diff for D6119: [12/n] Reconstruct ExecutionPlan from the ExecutionPlanSnapshot when it is available.
up
dgibson added inline comments to D6081: Remove pipeline_def from top-level SystemExecutionContext, only use in step execution contexts.
dgibson updated the diff for D6081: Remove pipeline_def from top-level SystemExecutionContext, only use in step execution contexts.
up
dgibson requested review of D6129: Skip event log watch tests on local mac runs.
Fri, Jan 22
Fri, Jan 22
dgibson added a reviewer for D6093: [8/n] Make StepInput.dagster_type a snapshot instead (no user code): alangenfeld.
dgibson retitled D6088: [6and7/n] remove solid from ExecutionStep from remove solid from ExecutionStep to [6and7/n] remove solid from ExecutionStep.
dgibson retitled D6086: [5/n] Wait to determine step compute_fn until you're actually in step execution from Wait to determine step compute_fn until you're actually in step execution to [5/n] Wait to determine step compute_fn until you're actually in step execution.
dgibson retitled D6084: [4/n] Remove mode_def from SystemExecutionContext, replace with mode_name from Remove mode_def from SystemExecutionContext, replace with mode_name to [4/n] Remove mode_def from SystemExecutionContext, replace with mode_name.
dgibson retitled D6083: [3/n] Include pipeline snapshot and execution plan snapshot on system execution context from Include pipeline snapshot and execution plan snapshot on system execution context to [3/n] Include pipeline snapshot and execution plan snapshot on system execution context.
dgibson requested review of D6095: [10/n]Make StepOutput user-code-free.
dgibson requested review of D6094: [9/n] Make StepInputSource use snapshots.
dgibson requested review of D6093: [8/n] Make StepInput.dagster_type a snapshot instead (no user code).
dgibson requested review of D6102: yield a SkipReason when a partitioned schedule can't find its partition.
dgibson updated the diff for D6088: [6and7/n] remove solid from ExecutionStep.
up
dgibson retitled D6088: [6and7/n] remove solid from ExecutionStep from remove solid from ExecutionStep to [7/n] remove solid from ExecutionStep.
Thu, Jan 21
Thu, Jan 21
dgibson requested review of D6088: [6and7/n] remove solid from ExecutionStep.
dgibson requested review of D6084: [4/n] Remove mode_def from SystemExecutionContext, replace with mode_name.
Wed, Jan 20
Wed, Jan 20
dgibson committed R1:0b046d20d7fb: Fix daemons not working in repo locations with more than one repo. (authored by dgibson).
Fix daemons not working in repo locations with more than one repo.
dgibson requested review of D6057: Fix daemons not working in repo locations with more than one repo..
dgibson requested changes to D6010: Run queuing enabled endpoint.
looks good, add test?
dgibson committed R1:83d28e88fab7: add error message when you start the daemon with no instance set. (authored by dgibson).
add error message when you start the daemon with no instance set.
dgibson requested review of D6056: add error message when you start the daemon with no instance set..
Unpin alembic
Tue, Jan 19
Tue, Jan 19
dgibson updated the summary of D6040: Unpin alembic.
dgibson requested review of D6040: Unpin alembic.
Add sensor_cli to api docs
dgibson requested review of D6038: Add sensor_cli to api docs.
Fri, Jan 15
Fri, Jan 15
dgibson committed R1:af02cdb364dc: Add retries to check in dagster=postgres init that checks table names (authored by dgibson).
Add retries to check in dagster=postgres init that checks table names
Fix deploy_docker example
dgibson requested review of D6028: Add retries to check in dagster=postgres init that checks table names.
dgibson requested review of D6027: Fix deploy_docker example.
Thu, Jan 14
Thu, Jan 14
dgibson committed R1:23b475873c91: Don't over-promise about daemon latency in the k8s docs (authored by dgibson).
Don't over-promise about daemon latency in the k8s docs
dgibson updated the diff for D6015: Don't over-promise about daemon latency in the k8s docs.
one more
dgibson updated the diff for D6015: Don't over-promise about daemon latency in the k8s docs.
regularly => periodically
dgibson published D6015: Don't over-promise about daemon latency in the k8s docs for review.
update link in RELEASING.md
dgibson requested review of D6006: update link in RELEASING.md.
Tweak daemon docs
dgibson published D6004: Tweak daemon docs for review.
Make instance migration error clearer
dgibson published D5991: make sure integrity error is caught within a connection contextmanager for review.
pass on run coordinator/queueing copy
dgibson committed R1:0b06d69e093c: Don't always run create_all during storage creation, add stamps to postgres… (authored by dgibson).
Don't always run create_all during storage creation, add stamps to postgres…
accepting assuming that any tests that fail due to this are also disabled
dgibson published D5995: Disable gRPC watch threads for 0.10.0 for review.