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 optimize debug #3

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

blaggacao
Copy link

  • feat: add debug statements
  • feat: reduce candiates over substituter instead of an inefficient map
  • fix: don't hardcode http2; http2 = enable should be authoritative
  • chore: bump to nix version 2.13.3

substituters are reduced over in order of their priority.
the previous map ended up behaving like a sequencial query instead of
a parallel one on big closures, so by reducing instead of mapping we
ensure not to incur in unecessary query work.
priorities should be set in such a way that the "closest" cache is also
set to the lowest priority, e.g. 1
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