Skip to content

Comments

Startup Performance And Security Improvements#61

Open
jwdotjs wants to merge 1 commit intomasterfrom
release-candidate/v5.0.0
Open

Startup Performance And Security Improvements#61
jwdotjs wants to merge 1 commit intomasterfrom
release-candidate/v5.0.0

Conversation

@jwdotjs
Copy link
Contributor

@jwdotjs jwdotjs commented Jun 24, 2018

See changelog for details

Trace

  • Previously Sendgrid, Pretty Error, Express and all related libraries were loaded during the initial require of Distraught, now they are required when first used (express isn't needed for the cron / worker servers, so this change should speed those processes up)

This looks like it makes about a ~100-200ms difference in initial startup time.

Old

screen shot 2018-06-23 at 6 23 15 pm

New

screen shot 2018-06-23 at 6 19 40 pm

@jwdotjs jwdotjs force-pushed the release-candidate/v5.0.0 branch from ce87e64 to c44eddd Compare June 24, 2018 01:17
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.

1 participant