User Details
User Details
- User Since
- Jul 6 2020, 12:49 PM (15 w, 1 d)
Today
Today
dgibson updated the diff for D4679: Change the default system for loading and executing user code to be gRPC instead of the CLI API.
rebase
dgibson committed R1:6d3e9aaac0af: Ensure that origins don't change when we switch from cli api repos to grpc repos (authored by dgibson).
Ensure that origins don't change when we switch from cli api repos to grpc repos
dgibson added inline comments to D4823: Ensure that origins don't change when we switch from cli api repos to grpc repos.
dgibson added a comment to D4821: RFC: [dagit] Start adding Workspace routes.
Overall this makes sense to me!
dgibson requested review of D4827: Append "_location" to default repository location name.
dgibson requested review of D4817: Remove executePlan dagster-graphql call.
dgibson requested review of D4824: Remove legacy repo format from loading path.
dgibson updated the diff for D4742: Remove legacy repo format from examples/tests, replace the default with the new standard format except when specifically testing legacy stuff.
remove the breaking part, just update teh examples and tests for now
dgibson added a reviewer for D4805: Don't rebuild the docker image every time you run a kind test locally: alangenfeld.
dgibson updated the diff for D4814: rm DagsterKubernetesPodOperator and test infra around it.
up
Yesterday
Yesterday
dgibson updated the diff for D4679: Change the default system for loading and executing user code to be gRPC instead of the CLI API.
up
dgibson requested review of D4814: rm DagsterKubernetesPodOperator and test infra around it.
test failures are transient/unrelated
dgibson requested review of D4813: Maintain backcompat in date_partition_range.
dgibson updated the diff for D4679: Change the default system for loading and executing user code to be gRPC instead of the CLI API.
up
dgibson committed R1:febffd667ebb: Use execute_step_with_structured_logs in DagsterDockerOperator (authored by dgibson).
Use execute_step_with_structured_logs in DagsterDockerOperator
dgibson planned changes to D4788: RFC: Add partitions to arbitrary cron-based schedules.
Sat, Oct 17
Sat, Oct 17
Fix helm namespace lifecycle in tests
Fri, Oct 16
Fri, Oct 16
dgibson requested review of D4805: Don't rebuild the docker image every time you run a kind test locally.
dgibson requested review of D4804: Fix helm namespace lifecycle in tests.
dgibson committed R1:69cb751f7166: [easy] Fix missing param change in date_partition_range call (authored by dgibson).
[easy] Fix missing param change in date_partition_range call
dgibson abandoned D4538: Write non-error execution logs to stdout instead of stderr, only surface stderr from child processes in host processes like the scheduler.
I'm going to hold off on this for now, will keep it in the back pocket though
dgibson published D4801: [easy] Fix missing param change in date_partition_range call for review.
Add execution_timezone field to schedules
dgibson committed R1:c52d2d09c3a3: Get cron schedule from ExternalSchedule rather than the Schedule DB (authored by dgibson).
Get cron schedule from ExternalSchedule rather than the Schedule DB
Make scheduler timezone-aware
dgibson updated the diff for D4725: Add execution_timezone field to schedules.
up
dgibson requested review of D4799: test termination in BK.
dgibson updated the diff for D4787: Require that start_date be on a month/day boundary for monthly/daily schedules.
up
dgibson updated the diff for D4725: Add execution_timezone field to schedules.
up
dgibson updated the diff for D4723: Get cron schedule from ExternalSchedule rather than the Schedule DB.
up
dgibson updated the diff for D4769: Use execute_step_with_structured_logs in DagsterDockerOperator.
up
dgibson abandoned D4796: Fix dagster_celery_docker.
rolling this into https://dagster.phacility.com/D4769
dgibson requested review of D4798: test for multiple trials with BK.
dgibson updated the diff for D4769: Use execute_step_with_structured_logs in DagsterDockerOperator.
.
dgibson updated the diff for D4796: Fix dagster_celery_docker.
now it is passing, check for flakiness?
dgibson requested review of D4769: Use execute_step_with_structured_logs in DagsterDockerOperator.
Thu, Oct 15
Thu, Oct 15
dgibson requested review of D4792: JUST BK FOR DAGSTER AIRFLOW DIFF.
dgibson added a comment to D4788: RFC: Add partitions to arbitrary cron-based schedules.
yeah, that would be a good future change I think. A missing piece is that we would need to be able to efficiently figure out the next partition without loading them all I think? Since you could have an hourly schedule that spans hundreds of days, for example.
dgibson updated the diff for D4723: Get cron schedule from ExternalSchedule rather than the Schedule DB.
up
dgibson published D4788: RFC: Add partitions to arbitrary cron-based schedules for review.