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

Add DatabricksPartitionSensor #30980

Merged
merged 23 commits into from
May 13, 2023
Merged

Conversation

harishkesavarao
Copy link
Contributor

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

@harishkesavarao
Copy link
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!

Copy link

@janvandervegt-db janvandervegt-db left a comment

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!

@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
@harishkesavarao harishkesavarao force-pushed the db-partition-sensor branch 3 times, most recently from 31b375f to b552c02 Compare May 11, 2023 15:02
@potiuk potiuk merged commit fdc7a31 into apache:main May 13, 2023
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