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

Modified datasource types don't get updated correctly #393

Closed
nthornton2010 opened this issue Feb 25, 2021 · 3 comments
Closed

Modified datasource types don't get updated correctly #393

nthornton2010 opened this issue Feb 25, 2021 · 3 comments

Comments

@nthornton2010
Copy link

When a datasource type is changed, it doesn't deploy correctly until the datasource's name is changed.

For example, I had 2 datsources that were both type AWS_LAMBDA but decided to go a different direction with both. 1 was updated to be type HTTP and the other type NONE. Neither of these started working until the datasource's name changed. The same happens when switching a non AWS_LAMBDA type to AWS_LAMBDA as well.

@nthornton2010
Copy link
Author

After further testing this morning, it looks like changing the data source's name doesn't always fix the issue. Sometimes the field needs to be manually removed from the schema and the original data source has to be manually deleted (both from the AWS console) before deploying. This behavior also causes issues with rolling back a deployment because you'd be switching types to go back which would also cause a simple rollback to break.

@nthornton2010
Copy link
Author

This might be related to the other issue I reported: #394

@nthornton2010
Copy link
Author

Based on the following comment from an AWS dev in a related issue and the source code for this plugin, I no longer believe this is an issue with this plugin but a design flaw in AppSync itself so I'm closing the issue. I say design flaw and not a bug because they say it's by design (a terrible design in my opinion).

aws/aws-appsync-community#146 (comment)

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

No branches or pull requests

1 participant