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

fix(gatsby-core-utils): multiple requests with different outputdir #35039

Merged
merged 2 commits into from
Mar 3, 2022

Conversation

wardpeet
Copy link
Contributor

@wardpeet wardpeet commented Mar 3, 2022

Description

Checking cache with mutex is a bit tricky as you mostly need to check things twice if you don't want other tasks to wait for the mutex. Now when 2 requests with the same url come in before one is already done we check the cache too and copy or return the cached path.

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Mar 3, 2022
@wardpeet wardpeet added topic: core Relates to Gatsby's core (e.g. page loading, reporter, state machine) and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Mar 3, 2022
pieh
pieh previously approved these changes Mar 3, 2022
@wardpeet wardpeet merged commit 2f361cc into master Mar 3, 2022
@wardpeet wardpeet deleted the fix/cached-fetch-remote branch March 3, 2022 15:04
LekoArts pushed a commit that referenced this pull request Mar 3, 2022
LekoArts added a commit that referenced this pull request Mar 9, 2022
@LekoArts
Copy link
Contributor

LekoArts commented Mar 9, 2022

Successfully published:

  • babel-plugin-remove-graphql-queries@4.9.1
  • babel-preset-gatsby@2.9.1
  • gatsby-cli@4.9.1
  • gatsby-core-utils@3.9.1
  • gatsby-link@4.9.1
  • gatsby-page-utils@2.9.1
  • gatsby-plugin-benchmark-reporting@2.9.1
  • gatsby-plugin-gatsby-cloud@4.9.1
  • gatsby-plugin-image@2.9.1
  • gatsby-plugin-manifest@4.9.1
  • gatsby-plugin-mdx@3.9.1
  • gatsby-plugin-offline@5.9.1
  • gatsby-plugin-page-creator@4.9.1
  • gatsby-plugin-preload-fonts@3.9.1
  • gatsby-plugin-sharp@4.9.1
  • gatsby-plugin-typescript@4.9.1
  • gatsby-remark-images@6.9.1
  • gatsby-source-contentful@7.7.2
  • gatsby-source-drupal@5.9.1
  • gatsby-source-filesystem@4.9.1
  • gatsby-source-graphql@4.9.1
  • gatsby-source-shopify@6.9.1
  • gatsby-source-wordpress@6.9.1
  • gatsby-telemetry@3.9.1
  • gatsby-transformer-remark@5.9.1
  • gatsby-transformer-sqip@4.9.1
  • gatsby@4.9.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: core Relates to Gatsby's core (e.g. page loading, reporter, state machine)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants