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

Replace all cross-dep. refs YAML scalars #3

Open
wants to merge 5 commits into
base: master
from

Conversation

@krismolendyke
Copy link

commented Aug 12, 2019

Based on #2, review, for clarity:

  • 57486bc hack multiple cross-dep. ref. replacement
  • 2ed6989 remove implicit BQ table dep. on dataset in same deployment
  • 13010ee Add explicit intra-deployment dependencies for tables to datasets

What

  • Replace all instances of $(out.x.y.z) cross-deployment references in values. Importantly, this includes YAML scalar string types, e.g., Big Query view query values.
  • Remove implicit dependency from BQ tables to datasets. This allows tables and views to make use of the cross-deployment refs w/o requiring that the datasets that they refer to be managed in the same deployment
  • Add support for establishing explicit dependencies from tables to datasets w/ a dependsOn key.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.