dbt-core
dbt-core copied to clipboard
add favor-state flag
Co-authored-by: josephberni [email protected]
resolves #5016
Description
Add --favor-state option which enables --defer to favor using --state node even if node exists in current target.
This optionally removes the below second --defer criterion when using the --favor-state flag.
Does the reference node exist as a database object in the current environment?
Checklist
- [ ] I have signed the CLA
- [x] I have run this code in development and it appears to resolve the stated issue
- [x] This PR includes tests, or tests are not required/relevant for this PR
- [x] I have added information about my change to be included in the CHANGELOG.
Currently in the process of signing the CLA.
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.
In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, don't hesitate to ping @drewbanin.
CLA has not been signed by users: @daniel-murray
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.
In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, don't hesitate to ping @drewbanin.
CLA has not been signed by users: @daniel-murray
@cla-bot check
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.
In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, don't hesitate to ping @drewbanin.
CLA has not been signed by users: @daniel-murray
The cla-bot has been summoned, and re-checked this pull request!
Just adding a big plus one to getting this in! Any update @daniel-murray ? (Sorry for double post on issue too!)
@daniel-murray: Is this something that you'd like to move forward?
CLA has been signed off, so hoping we can get this merged soonish! Let me know if you need anything else from me.
The first thing to do is to merge the current state of main, resolve the differences, and update.
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.
In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, don't hesitate to ping @drewbanin.
CLA has not been signed by users: @daniel-murray
@josephberni I think we might need @daniel-murray to sign the CLA to get this merged
I still want this real bad :D - any movement here @josephberni ?
I've created a new branch for this change where I am now the owner https://github.com/dbt-labs/dbt-core/pull/5859
Let me know if there is anything else you need from me & sorry @NiallRees it took so long!
closing in favor of https://github.com/dbt-labs/dbt-core/pull/5859