search

Snowflake-Labs / OpenLineage-AccessHistory-Setup

Guideline to extract table lineage info in OpenLineage format from access history view
Apache License 2.0
10 stars 5 forks source link

Import days_ago #7

Closed Sheeri closed 2 years ago

Sheeri commented 2 years ago

fixes #6

Tested locally, the DAG import error went away when I added this import line.

rossturk commented 2 years ago

Ah, the issue here is that days_ago shouldn't be part of the DAG declaration. Apparently, this is a practice that the Airflow folks have been trying to stamp out for a while. An explicit start date should be used instead, like start_date=datetime(2022, 4, 12).

I believe @josh-fell probably missed this one when he was removing all of the instances of days_ago in a previous PR. I'll submit a separate PR with this fix.

rossturk commented 2 years ago

I've submitted #19, which removes days_ago from this DAG. I believe this PR can be closed now w/o merging.