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

airflow-provider-vdk: Example DAG #847

Merged
merged 3 commits into from
Jun 1, 2022
Merged

airflow-provider-vdk: Example DAG #847

merged 3 commits into from
Jun 1, 2022

Conversation

gageorgiev
Copy link
Contributor

This change includes an example DAG demonstrating
how to use the VDKOperator to run a data job through
an Airflow DAG.

Testing done: executed through Airflow UI against
deployed control-service successfully

Signed-off-by: Gabriel Georgiev gageorgiev@vmware.com

This change includes an example DAG demonstrating
how to use the VDKOperator to run a data job through
an Airflow DAG.

Testing done: executed through Airflow UI against
deployed control-service successfully

Signed-off-by: Gabriel Georgiev <gageorgiev@vmware.com>
gageorgiev and others added 2 commits May 31, 2022 12:19
Signed-off-by: Gabriel Georgiev <gageorgiev@vmware.com>
Copy link
Contributor

@doks5 doks5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gageorgiev gageorgiev merged commit f649d1a into main Jun 1, 2022
@gageorgiev gageorgiev deleted the person/gageorgiev/dag branch June 1, 2022 06:21
)

job2 = VDKOperator(
conn_id="tms-example_vdk_connection",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why different conn ids?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was a mistake that I've fixed and forgot to push.

from airflow import DAG
from vdk_provider.operators.vdk import VDKOperator

with DAG(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's add comments explaining what is going on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants