Skip to content

Add DatabricksPartitionSensor#30980

Merged
potiuk merged 23 commits into
apache:mainfrom
harishkesavarao:db-partition-sensor
May 13, 2023
Merged

Add DatabricksPartitionSensor#30980
potiuk merged 23 commits into
apache:mainfrom
harishkesavarao:db-partition-sensor

Conversation

@harishkesavarao

Copy link
Copy Markdown
Contributor

PR for Databricks Partition Sensor.
Note: Previous comments via PR #28950 have been addressed.

@harishkesavarao

Copy link
Copy Markdown
Contributor Author

@eladkal @o-nikolas @josh-fell this is a follow up of my previous PR #28950 where I intended to submit 3 separate PRs for each sensor. I have addressed comments in the older PR and created this PR for just the partition sensor (previously merged the SQL sensor via #30477).
It would be great if you can review the PR, thank you for your time!

@janvandervegt-db janvandervegt-db left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Some small pieces of feedback but looks good!

Comment thread airflow/providers/databricks/sensors/databricks_partition.py Outdated
Comment thread docs/apache-airflow-providers-databricks/operators/sql.rst Outdated
Comment thread docs/apache-airflow-providers-databricks/operators/sql.rst Outdated
Comment thread docs/apache-airflow-providers-databricks/operators/sql.rst Outdated
@harishkesavarao
harishkesavarao force-pushed the db-partition-sensor branch 4 times, most recently from 10e68e3 to 50b8ab3 Compare May 7, 2023 20:33
@eladkal eladkal changed the title Db Partition Sensor Add DatabricksPartitionSensor May 8, 2023
@eladkal
eladkal requested review from josh-fell and o-nikolas May 8, 2023 06:58
@harishkesavarao
harishkesavarao force-pushed the db-partition-sensor branch 3 times, most recently from 31b375f to b552c02 Compare May 11, 2023 15:02
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.

3 participants