Skip to content

Reduce FeatureBench execute-phase heap churn - #207

Open
shaharuk-yb wants to merge 2 commits into
mainfrom
featurebench-heap-optimizations
Open

Reduce FeatureBench execute-phase heap churn#207
shaharuk-yb wants to merge 2 commits into
mainfrom
featurebench-heap-optimizations

Conversation

@shaharuk-yb

Copy link
Copy Markdown

Free result sets promptly and cut avoidable per-transaction garbage in the execute hot loop, reducing GC pressure during long/high-concurrency runs.

  • Close ResultSets via try-with-resources in the execute hot loop, in runExplainAnalyse, and in the pg_stat collectors, so buffers are released promptly instead of lingering on cached PreparedStatements.
  • Remove the dead pg_prepared_statements collection (was flagged TODO-remove and duplicated into every query's JSON entry).
  • Precompute Query.isReturningQuery once instead of calling PreparedStatement.toString() on every execution.
  • Allocate the expression-context map in generateParameterValues lazily, avoiding a throwaway HashMap per execution for queries without reference/expression bindings.

automated · Claude Code (Opus 4.8)

shaharuk-yb and others added 2 commits July 13, 2026 14:28
Free result sets promptly and cut avoidable per-transaction garbage in the
execute hot loop, reducing GC pressure during long/high-concurrency runs.

- Close ResultSets via try-with-resources in the execute hot loop, in
  runExplainAnalyse, and in the pg_stat collectors, so buffers are released
  promptly instead of lingering on cached PreparedStatements.
- Remove the dead pg_prepared_statements collection (was flagged TODO-remove and
  duplicated into every query's JSON entry).
- Precompute Query.isReturningQuery once instead of calling
  PreparedStatement.toString() on every execution.
- Allocate the expression-context map in generateParameterValues lazily, avoiding
  a throwaway HashMap per execution for queries without reference/expression bindings.

---
_automated · Claude Code (Opus 4.8)_

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@shaharuk-yb
shaharuk-yb requested a review from yashtyagi-yb July 13, 2026 09:05
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.

1 participant