proxysql

Commit Graph

Author	SHA1	Message	Date
Rene Cannao	7f5bbf0f19	Fix compilation error: add missing #include <cstdint> in MonitorHealthDecision.cpp The int64_t cast in can_unshun_server() requires <cstdint>, which was not included, causing build failure. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>	2 months ago
René Cannaò	99980ca196	Address review feedback on monitor health (PR #5507 ) - Fix integer overflow: cast shun_recovery_time_sec to int64_t before multiplying by 1000 to prevent overflow for large values (>24 days) - Add docstring note about production lag==-2 override path (issue #959) not covered by this simplified extraction - Remove trailing "fix plan" comment from test	2 months ago
René Cannaò	59654e48a3	Extract monitor health decision logic into pure functions (Phase 3.3, #5491 ) New files: - include/MonitorHealthDecision.h: declarations for 4 pure functions - lib/MonitorHealthDecision.cpp: implementations Extracted functions: - should_shun_on_connect_errors(): mirrors MySrvC::connect_error() threshold logic (min(shun_on_failures, connect_retries+1)) - can_unshun_server(): mirrors MyHGC recovery loop with time check, connect_timeout_max cap, min 1s floor, and kill_all drain check - should_shun_on_replication_lag(): mirrors MySQL_HostGroups_Manager lag check with consecutive count threshold - can_recover_from_replication_lag(): mirrors lag recovery check All functions are pure (no global state, no I/O) and added to libproxysql.a via lib/Makefile.	2 months ago

Author

SHA1

Message

Date

Rene Cannao

7f5bbf0f19

Fix compilation error: add missing #include <cstdint> in MonitorHealthDecision.cpp

The int64_t cast in can_unshun_server() requires <cstdint>, which was not
included, causing build failure.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

René Cannaò

99980ca196

Address review feedback on monitor health (PR #5507 )

- Fix integer overflow: cast shun_recovery_time_sec to int64_t before
  multiplying by 1000 to prevent overflow for large values (>24 days)
- Add docstring note about production lag==-2 override path (issue #959)
  not covered by this simplified extraction
- Remove trailing "fix plan" comment from test

René Cannaò

59654e48a3

Extract monitor health decision logic into pure functions (Phase 3.3, #5491 )

New files:
- include/MonitorHealthDecision.h: declarations for 4 pure functions
- lib/MonitorHealthDecision.cpp: implementations

Extracted functions:
- should_shun_on_connect_errors(): mirrors MySrvC::connect_error()
  threshold logic (min(shun_on_failures, connect_retries+1))
- can_unshun_server(): mirrors MyHGC recovery loop with time check,
  connect_timeout_max cap, min 1s floor, and kill_all drain check
- should_shun_on_replication_lag(): mirrors MySQL_HostGroups_Manager
  lag check with consecutive count threshold
- can_recover_from_replication_lag(): mirrors lag recovery check

All functions are pure (no global state, no I/O) and added to
libproxysql.a via lib/Makefile.

3 Commits (d5b796cdea6045b18abf7f98415478e8ffaaa3ca)