3 commits

Author	SHA1	Message	Date
Graham McIntire	d9c04f7e0c	Fix code review findings: security, correctness, performance, dead code ... Security: - Add SQL identifier validation to DbOptimizer.analyze_table/vacuum_table - Move health/readiness routes to non-rate-limited pipeline Correctness: - Fix monitor reference leak in StreamingPacketsPubSub (track refs, demonitor on unsubscribe) - Persist circuit breaker half-open state transitions - Simplify ShutdownHandler.terminate to avoid pointless Process.send_after Performance: - Batch marker removal into single WebSocket event (remove_markers_batch) - Use Task.Supervisor.start_child for fire-and-forget broadcasts Dead code removal: - Remove copy_insert/regular_batch_insert/rows_to_csv from DbOptimizer - Remove dead packet_pipeline_integration tests - Remove unused telemetry event detach Other: - Fix Callsign.valid? docstring to match permissive behavior - Fix DatabaseMetrics to delegate instead of hardcoding pool stats - Add circuit breaker and encoding test coverage	2026-02-19 18:49:27 -06:00
Graham McIntire	60efa7677b	Fix ShutdownHandler terminating during normal startup ... - Only initiate graceful shutdown for abnormal termination reasons - Ignore :shutdown, {:shutdown, _}, and :normal termination reasons - Add logging for SIGTERM handler installation Co-Authored-By: Claude <noreply@anthropic.com>	2025-07-26 15:55:56 -05:00
Graham McIntire	eeeebef34e	Fix PgBouncer database connection and graceful shutdown ... - Add SKIP_DB_CREATE env var to skip database creation when using PgBouncer - Fix redis_children() to return proper child specs - Implement zero-downtime graceful shutdown without user notifications - Add health check plug for Kubernetes probes - Remove user-facing shutdown notifications for seamless deployments Co-Authored-By: Claude <noreply@anthropic.com>	2025-07-26 15:51:06 -05:00