Skip to content

Improve query and UI performance - #75

Merged
shobhit-seg merged 1 commit into
mainfrom
codex/improve-clicky-performance
Jul 25, 2026
Merged

shobhit-seg merged 1 commit into
mainfrom
codex/improve-clicky-performance

Conversation

@shobhit-seg

Copy link
Copy Markdown
Contributor

Summary

  • reuse ClickHouse result metadata to eliminate redundant DESCRIBE TABLE requests
  • cache and debounce table search, reduce render-loop work, and optimize result rendering
  • close stale and redundant clients, extend default limits to CTE selects, and remove the query Format button

Measured impact

  • query task: 4.956 ms to 2.505 ms in the controlled latency benchmark
  • Explorer fetch: 4.978 ms to 2.500 ms
  • six rapid table-search keystrokes: six rebuilds to one
  • idle resizer theme bindings over 600 frames: 6000 to zero

Validation

  • 119 tests passed
  • Ruff lint and format checks passed
  • compileall and git diff --check passed
  • DearPyGUI toolbar and row-clipper smoke tests passed

@abhi204

abhi204 commented Jul 24, 2026

Copy link
Copy Markdown

🤖 AI pre-review — posted by Claude Code on behalf of @abhi204 (model: claude-opus-4-8). Not a formal review; @abhi204 still signs off.

No blocking issues found. Checked for bugs, security, and CLAUDE.md compliance.

Reviewed the connection-lifecycle rework in database.py (client close paths, lock-guarded disconnect, pool race handling, close_all), the resizer theme state-change optimization and CTE-aware _add_default_limit in query_interface.py, the result-metadata reuse (get_result_column_types), and the table-list caching/debounce in table_browser_ui.py. All changes are correct and well-covered by the added tests.

@shobhit-seg
shobhit-seg added this pull request to the merge queue Jul 25, 2026
Merged via the queue into main with commit b6c5878 Jul 25, 2026
1 check passed
@shobhit-seg
shobhit-seg deleted the codex/improve-clicky-performance branch July 25, 2026 10:07
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.

3 participants