Fixing #5238 - SQL Issues due to non-visible chars #5240
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Searching for Poller Items Generates SQL Errors
* On large systems statistics gathering at the beginning of a poller run lead to excessive polling times * It's important to remove all items from the poller_output table for the poller as it impacts the end of the previous poller * The other change here has to do with large databases where the poller_output memory table can get pushed into swap. * Moved statistics out of the internal loop, and made a few optimizations for the $totals query to perform better. * We only care about total_ports if the snmp_port is > 0
|
Good comment. I'm still trying to figure out why I'm having poller issues. Made another discovery this morning. Going to add one more minor change to this file. I'm up for a setting for this as I have no choice but to have this behavior on my 10.5.13 instance. The darn memory tables are being pushed to swap. |
|
@netniV, I think a setting is the right way to go per @browniebraun comments. |
This is a compromise with regard to a possible issue with MariaDB 10.3 to only allow this via setting, and only allow for single poller systems. We will look to re-design this in a future Cacti release.
This test added multi-output values that return a NaN or 'U'.
Edit Graph Template link missing base_url.
Template Export missing Graph Template columns multiple and test_source
netniV
approved these changes
Feb 20, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Searching for Poller Items Generates SQL Errors