puppetlabs / puppetdb

Centralized Puppet Storage
http://docs.puppetlabs.com/puppetdb
Apache License 2.0
299 stars 225 forks source link

(PE-37466) Ensure query monitor forgets for all relevant routes #3933

Closed rbrw closed 8 months ago

CLAassistant commented 8 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: austb
:x: rbrw
You have signed the CLA already but the status is still pending? Let us recheck it.

rbrw commented 8 months ago

Suggest ignoring blankspace when looking at some of the commits.

jonathannewman commented 8 months ago

Given the relative complexity of the code, and the multi-threaded nature of it, have you considered adding debug and trace level logging to both this code and the query monitor to help isolate issues when they occur?