Skip to content

feat: Allow use of backend that is not in list of destinations#13475

Merged
kodiakhq[bot] merged 6 commits intomainfrom
allow-backend-that-is-not-destination
Aug 30, 2023
Merged

feat: Allow use of backend that is not in list of destinations#13475
kodiakhq[bot] merged 6 commits intomainfrom
allow-backend-that-is-not-destination

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

Discovered while writing the documentation for backend options: we should be able to support using a backend that is different from the destination being synced to, but currently this doesn't work.

Without the change in this PR, destinations like BigQuery and Snowflake wouldn't be able to use state with incremental tables. With this change, it can be achieved by using a different destination as the backend (e.g. PostgreSQL, SQLite, or any other that supports overwrite mode)

@hermanschaaf hermanschaaf requested review from erezrokah and removed request for yevgenypats August 30, 2023 12:12
@cq-bot cq-bot added the cli label Aug 30, 2023
@hermanschaaf
Copy link
Copy Markdown
Contributor Author

Had to make some adjustments to the variable replacement code, but the tests are now passing on Windows as well

@hermanschaaf hermanschaaf added the automerge Automatically merge once required checks pass label Aug 30, 2023
@kodiakhq kodiakhq bot merged commit 1fea1f8 into main Aug 30, 2023
@kodiakhq kodiakhq bot deleted the allow-backend-that-is-not-destination branch August 30, 2023 14:22
kodiakhq bot pushed a commit that referenced this pull request Aug 30, 2023
🤖 I have created a release *beep* *boop*
---


## [3.16.0](cli-v3.15.1...cli-v3.16.0) (2023-08-30)


### Features

* Allow use of backend that is not in list of destinations ([#13475](#13475)) ([1fea1f8](1fea1f8))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants