Skip to content

select instances aka databases with one query - #1657

Closed
dgf-hogrefe wants to merge 2 commits into
puppetlabs:mainfrom
hogrefe-digital:select-instances
Closed

dgf-hogrefe wants to merge 2 commits into
puppetlabs:mainfrom
hogrefe-digital:select-instances

Conversation

@dgf-hogrefe

Copy link
Copy Markdown

In a server environment with thousands of databases, the original iterative retrieval of database information is extremely slow and results in a Puppet run taking minutes to complete. This change replaces these queries with a single one by querying all relevant attributes from the information schema.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@jst-cyr

jst-cyr commented Aug 28, 2026

Copy link
Copy Markdown
Member

Hey there @dgf-hogrefe , I'm sorry there hasn't been any activity here yet. I'm reviewing old PRs and am wondering if this is still an issue you are seeing?

@jst-cyr jst-cyr added the cancelled Closed/Cancelled due to inactivity or lack of need label Sep 11, 2026
@jst-cyr

jst-cyr commented Sep 11, 2026

Copy link
Copy Markdown
Member

Closing for now. If a need comes up in the future, we can reopen.

@jst-cyr jst-cyr closed this Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cancelled Closed/Cancelled due to inactivity or lack of need

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants