-
Notifications
You must be signed in to change notification settings - Fork 15
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
New playbook format which separates database target from SQL #149
Comments
alexanderdean
changed the title
New playbook format which separates database target
New playbook format which separates database target from SQL
Jul 17, 2020
I like this. A good example of it done well in great_expectations |
Right - that's perfect... Once we break these out, it becomes safe to come up with a .jar style format which creates a zipped bundle containing all the files (but not the database creds!). |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This would be a better abstraction as really the SQL scripts are an intrinsic part of the job-to-be-done (the WHAT) but the target properties/credentials (the WHERE) are orthogonal to this.
The text was updated successfully, but these errors were encountered: