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

PROXYSQL FLUSH LOGS forces debug database flush #4833

Open
wants to merge 12 commits into
base: v3.0
Choose a base branch
from

Conversation

renecannao
Copy link
Contributor

Running PROXYSQL FLUSH LOGS will also trigger the flushing of the debug database , if enabled.

Also increased batching size from 31 to 100

Also increased barching size from 31 to 100
Error handling for 'sqlite3_step' should be performed in a similar
fashion as for 'sqlite3_exec'. Not handling these potential errors could
result in invalid empty/incomplete resultsets when reading from the
internal SQLite3 databases.
This new entry allows to associate session, thread and query being
processed, and offers some extra details about the query itself.
- Fixed issue with SQLite3 helper functions. Invalid use of
  'sqlite3_step' lead to fixing issue in ProxySQL itself.
- Fixed logic issues on tests checking duplicated conditions.
- Improved waiting logic checking for new entries in debug database.
- Added new utility functions for SQLite3 databases access (debug
  database) following MySQL utilities conventions.
- Simplified interface in several utilities functions with new, and more
  explicit naming convention.
Previously 'test_auth_methods' flagged a current ProxySQL limitation
regarding auth full-handshakes with an error. This has been replaced
with a warning until fixed.
By default 'get_proxysql_cpu_usage' now uses the currently configured
interval for measuring system CPU usage ('admin-stats_system_cpu'). A
parameter still allows to use a custom configured interval, but since
maximum resolution is used for testing environments, the default value
should normally be the one used.
…lgorithm-t.cpp'

More queries were received during testing ('stats' user) that matched
the previous check criteria. Warmup queries should now be uniquely
identified with the test itself.
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.

2 participants