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

Resolver Tweaks #2304

Merged
merged 4 commits into from Jun 7, 2018

Conversation

Projects
1 participant
@techalchemy
Member

techalchemy commented Jun 5, 2018

Flamegraphs tell me that json resolution failures are incredibly slow and bad for performance

  • Disable Json resolver
  • Tweak vcs resolution

techalchemy added some commits Jun 5, 2018

Disable json resolver and tweak vcs resolution
Signed-off-by: Dan Ryan <dan@danryan.co>
Fix resolver bug
Signed-off-by: Dan Ryan <dan@danryan.co>
Create temp src dir if not using environment
Signed-off-by: Dan Ryan <dan@danryan.co>

@techalchemy techalchemy merged commit a9461cc into master Jun 7, 2018

2 checks passed

buildkite/pipenv Build #289 passed (10 minutes, 12 seconds)
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@techalchemy techalchemy moved this from Done to Needs Changelog in 2018.06.x Release Jun 16, 2018

@techalchemy techalchemy moved this from Needs Changelog to Done in 2018.06.x Release Jun 16, 2018

@techalchemy techalchemy deleted the tweak-resolver branch Jun 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment