Skip to content

Conversation

fusionstrings
Copy link
Owner

@fusionstrings fusionstrings commented Dec 3, 2021

Deploy as a single edge function with objective of ~50ms SSR response.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 3, 2021

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 427fc0a
Status: ✅  Deploy successful!
Preview URL: https://bd8756d3.jspm-packages.pages.dev

View logs

@fusionstrings fusionstrings marked this pull request as draft December 3, 2021 15:25
@fusionstrings fusionstrings marked this pull request as ready for review December 6, 2021 01:36
@guybedford
Copy link

I'm still getting 1s on cold loads - I wonder if workers are faster? It's all well and good to use the latest hype, but over 500ms for a page request is too slow.

@fusionstrings
Copy link
Owner Author

I'm still getting 1s on cold loads - I wonder if workers are faster? It's all well and good to use the latest hype, but over 500ms for a page request is too slow.

would you suggest something else? I can try other solutions.

const NPM_PROVIDER_URL = "https://ga.jspm.io/npm:";
const packageName = params.package.join("/");
const baseURL = `${NPM_PROVIDER_URL}${packageName}`;
const jspmPackage = await fetch(

Choose a reason for hiding this comment

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

Can you do this fetch in parallel with the readme fetch?

Copy link
Owner Author

Choose a reason for hiding this comment

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

@guybedford
Copy link

I don't think Rust would result in any performance benefit - if anything it increases the startup time, unless there is an actual computation bottleneck. Rather we just need to find an edge that is close to Amazon cloudfront. Possibly comparing Amazon lambda might be another option.

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.

2 participants