Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[dagster-airlift][federation-apis] Standalone sensor def creation fxn #25862

Open
wants to merge 1 commit into
base: dpeng817/load_airflow_dag_asset_specs
Choose a base branch
from

Conversation

dpeng817
Copy link
Contributor

@dpeng817 dpeng817 commented Nov 12, 2024

Summary & Motivation

Previously our fxn was creating a Definitions object. Let's change to creating a SensorDefinition directly.

Copy link
Contributor Author

dpeng817 commented Nov 12, 2024

@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch from 292d67f to 5f446bf Compare November 13, 2024 18:59
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from 8613eae to ecd505a Compare November 13, 2024 18:59
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch from 5f446bf to 23aea39 Compare November 13, 2024 19:09
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from ecd505a to 78b32c8 Compare November 13, 2024 19:09
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch from 23aea39 to 60edc68 Compare November 13, 2024 19:14
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from 78b32c8 to e23ec9c Compare November 13, 2024 19:14
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch from 60edc68 to 292239c Compare November 13, 2024 19:58
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from e23ec9c to ef3869a Compare November 13, 2024 19:58
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch from 292239c to 8196ec1 Compare November 13, 2024 20:00
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from ef3869a to a68a877 Compare November 13, 2024 20:00
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch 2 times, most recently from b990c7f to 2291ada Compare November 13, 2024 21:59
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from a68a877 to fd90c6c Compare November 13, 2024 21:59
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch from 2291ada to 5706484 Compare November 13, 2024 22:02
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from fd90c6c to f1ca203 Compare November 13, 2024 22:02
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch from 5706484 to f2f5180 Compare November 13, 2024 22:17
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from f1ca203 to 7fb9149 Compare November 13, 2024 22:18
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch from f2f5180 to 744d696 Compare November 13, 2024 23:18
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from 7fb9149 to e22695f Compare November 13, 2024 23:18
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch from 744d696 to c279577 Compare November 13, 2024 23:43
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from e22695f to 52fe220 Compare November 13, 2024 23:43
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch 2 times, most recently from e1f9602 to e9bf723 Compare November 13, 2024 23:46
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from 52fe220 to b712824 Compare November 13, 2024 23:46
@dpeng817 dpeng817 force-pushed the dpeng817/load_airflow_dag_asset_specs branch from e9bf723 to 3117e08 Compare November 14, 2024 19:45
@dpeng817 dpeng817 force-pushed the dpeng817/standalone_sensor_def_fxn branch from b712824 to cc5370b Compare November 14, 2024 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-to-migrate Docs to migrate to new docs site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants