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.

Sign up for GitHub

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

Jump to bottom

Use a single FacterDB.get_facts call #158

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Use a single FacterDB.get_facts call #158

wants to merge 2 commits into from

Commits on Sep 13, 2024

  1. Use a single FacterDB.get_facts call 

    The major cost is FacterDB.get_facts. Instead of calling it once for
every OS version this instead calls it once with a complex filter. It
then uses JGrep to simulate another smaller DB and futher query that.

In my non-scientific test this reduces the load time of puppet-example
from 3.5 to 2.5 seconds.
    @ekohl
    ekohl committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    5fcc51c View commit details
    Browse the repository at this point in the history

  2. Use FacterDB.filter_results

    @ekohl
    ekohl committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    70f4c33 View commit details
    Browse the repository at this point in the history