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.

Sign up for GitHub

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

Jump to bottom

Feature: Databricks compatibility #25

Merged
merged 6 commits into from
Jun 6, 2023
Merged

Feature: Databricks compatibility #25

merged 6 commits into from
Jun 6, 2023

Conversation

fivetran-catfritz
Copy link
Contributor

@fivetran-catfritz fivetran-catfritz commented May 17, 2023 •
edited
Loading

Confirm the following files were correctly updated automatically:

  • hooks/pre-command line added
  • pipeline.yml section added
  • sample.profiles.yml catalog updated
  • integration_tests/dbt_project.yml section added (check for dupes though)
  • Version updates for dbt_project.yml and integration_tests/dbt_project.yml (automatically moved to next breaking change)
  • Changelog (automatically moved to next breaking change, but check the PR links were added correctly!)
  • greenhouse_source.yml database logic updated

Manual updates are:

  • README installation and dependencies
  • [NA] Incremental Models
  • Regen Docs
  • Local Databricks run and test passing
  • Buildkite passing

This was linked to issues May 31, 2023
[Feature] Add postgres capability #22
Closed
[Feature] Databricks Compatibility #24
Closed
@fivetran-catfritz
Copy link
Contributor Author

Standard updates.

@fivetran-catfritz fivetran-catfritz self-assigned this Jun 1, 2023
fivetran-joemarkiewicz
fivetran-joemarkiewicz approved these changes Jun 5, 2023
View reviewed changes
Copy link
Contributor

@fivetran-joemarkiewicz fivetran-joemarkiewicz left a comment

Choose a reason for hiding this comment

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

LGTM!

@fivetran-catfritz fivetran-catfritz merged commit 4c98b01 into main Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fivetran-joemarkiewicz fivetran-joemarkiewicz fivetran-joemarkiewicz approved these changes

Assignees

@fivetran-catfritz fivetran-catfritz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature] Databricks Compatibility [Feature] Add postgres capability
2 participants
@fivetran-catfritz @fivetran-joemarkiewicz