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

[Request] Update of apollo-cache-inmemory to 1.3.12 #334

Closed
misterbridge opened this issue Jan 8, 2019 · 3 comments
Closed

[Request] Update of apollo-cache-inmemory to 1.3.12 #334

misterbridge opened this issue Jan 8, 2019 · 3 comments
Assignees
Labels
investigating Amplify is looking into the issue

Comments

@misterbridge
Copy link

Do you want to request a feature or report a bug?
Update of dependency package for bug fix

What is the current behavior?
Memory leak in optimistic response

Thank you to update apollo-cache-inmemory to 1.3.12
apollographql/apollo-client#4210

@haverchuck haverchuck added the investigating Amplify is looking into the issue label Jan 14, 2019
@jrounsav
Copy link

jrounsav commented May 6, 2019

This yarn feature was new to me so I'm posting here.
We couldn't wait on the PR. Added the following to our package.json and ran yarn install to override the dependency.

"resolutions": {
    "apollo-cache-inmemory": "1.3.12"
  }

@biker2000on
Copy link

This is still a major issue in using Appsync Client. When running multiple mutations and writing to the cache with Optimistic Response, the app quickly runs out of memory and Chrome stops with the out of memory error eventually shutting down the page.

As mentioned above, the dependency for apollo-cache-inmemory needs to be updated to at least 1.3.12 to mitigate the issue.

apollographql/apollo-client#4210

@misterbridge
Copy link
Author

Looks like fixed, thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigating Amplify is looking into the issue
Projects
None yet
Development

No branches or pull requests

5 participants