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.

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
1 participant