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

changed primary key to non-computed #95

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bmschow
Copy link

@bmschow bmschow commented Apr 26, 2024

This PR obviously changes the code generated by Speakeasy, so someone will have to modify the speakeasy config correctly to make this change (I do not have access to speakeasy to confirm that any changes will work). However, this fixes #91

Recreating the issue:
Create a postgres source that reads from a table that has a defined primary key and use "sync_mode": "incremental_deduped_history" - do not supply a primary key here. This should be able to be applied. However, if you make an changes to this resource, terraform will read and try to send the primary key that was supplied by the database causing the API error primary key for stream: <stream> already pre-defined. Please do NOT include a primary key configuration for this stream and will not be able to apply the change.

By changing primary_key in streams to non-computed, we avoid this error.

@jduckwo
Copy link

jduckwo commented Jun 27, 2024

Any update on getting this resolved one way or another? We are running into a similar issue and it is a blocker for us to use the terraform provider for AirByte.

@abusutil
Copy link

abusutil commented Aug 5, 2024

We facing the same issue, and it's very difficult to use the provider when we want to update a existing connection.

Do you have any news ?

@jduckwo
Copy link

jduckwo commented Aug 5, 2024

@abusutil We just decided to pivot and use the API since it gave us more control.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Terraform Provider Conflict When Primary Key Is Defined by Source
3 participants