Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

axe312ger
Copy link
Contributor

  • increase default pageLimit to 1000
  • add backoff logic that retries with a lower pageLimit till we are able to sync data from Contentful

The first commit in here should be replaced by #30257

@axe312ger axe312ger added the status: needs core review Currently awaiting review from Core team member label Mar 23, 2021
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Mar 23, 2021
@LekoArts LekoArts added topic: source-contentful Related to Gatsby's integration with Contentful and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Mar 25, 2021
@axe312ger axe312ger force-pushed the feat/contentful-page-limit-backoff branch from 69789c9 to b6a7dd5 Compare March 29, 2021 08:50
Copy link
Contributor

@wardpeet wardpeet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small comments

@axe312ger
Copy link
Contributor Author

@wardpeet done

@axe312ger axe312ger force-pushed the feat/contentful-page-limit-backoff branch from a49c874 to 4fc1237 Compare March 29, 2021 16:25
Copy link
Contributor

@wardpeet wardpeet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@wardpeet wardpeet changed the title Perf: Increase Contentful sync by up to 10x feat(gatsby-source-contentful): Increase Contentful sync by up to 10x Mar 30, 2021
@wardpeet wardpeet merged commit b9791fe into master Mar 30, 2021
@wardpeet wardpeet deleted the feat/contentful-page-limit-backoff branch March 30, 2021 08:50
vladar pushed a commit that referenced this pull request Mar 30, 2021
vladar pushed a commit that referenced this pull request Mar 30, 2021
…#30422) (#30571)

Co-authored-by: Ward Peeters <[email protected]>
(cherry picked from commit b9791fe)

Co-authored-by: Benedikt Rötsch <[email protected]>
pieh pushed a commit that referenced this pull request Apr 2, 2021
pieh pushed a commit that referenced this pull request Apr 2, 2021
…#30422) (#30643)

Co-authored-by: Ward Peeters <[email protected]>
(cherry picked from commit b9791fe)

Co-authored-by: Benedikt Rötsch <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs core review Currently awaiting review from Core team member topic: source-contentful Related to Gatsby's integration with Contentful
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants