Skip to content

Add warning log to EvictionQueue::Purge for slow purge detection#44

Open
krleonid wants to merge 4 commits into
v1.5-variegatafrom
duckdb_log_variegata
Open

Add warning log to EvictionQueue::Purge for slow purge detection#44
krleonid wants to merge 4 commits into
v1.5-variegatafrom
duckdb_log_variegata

Conversation

@krleonid
Copy link
Copy Markdown
Owner

Summary

  • Add DUCKDB_LOG_WARNING to EvictionQueue::Purge() when iterations > 10 or elapsed time > 1 second
  • Logs queue_size and dead_nodes count for diagnosis
  • Also prints to stderr in DEBUG builds via Printer::PrintF

Usage

SET GLOBAL enable_logging = true;
SET GLOBAL logging_level = 'warning';
-- run workload, then:
SELECT * FROM duckdb_logs() WHERE message LIKE '%Purge%';

Test plan

  • Run concurrent workload with multiple attached databases
  • Verify warning appears in duckdb_logs() when purge takes many iterations
  • Verify no output when purge is fast (< 10 iterations, < 1s)

🤖 Generated with Claude Code

@krleonid krleonid force-pushed the duckdb_log_variegata branch from 8fddd18 to 729195d Compare May 12, 2026 09:43
EtgarDev and others added 4 commits May 12, 2026 12:47
- Suppress WAL auto-checkpoint during pragma_storage_info section to prevent
  vacuum merging transient segments before they can be inspected
- Skip test under force_storage_restart (per-statement restarts checkpoint
  after every statement, inherently merging transient segments)
- Remove empty skip_tests from adaptive config (caused run_tests.py crash)
- Add adaptive_segment_allocation.json to Main.yml CI pipeline
…ed > 1s

Logs via DUCKDB_LOG_WARNING with queue_size and dead_nodes count.
Also prints to stderr in DEBUG builds for local debugging.

Requires: SET GLOBAL enable_logging = true; SET GLOBAL logging_level = 'warning';

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@krleonid krleonid force-pushed the duckdb_log_variegata branch from 729195d to 3a67435 Compare May 12, 2026 09:47
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