Toilal / rebulk

98%

master: 97%

LAST BUILD ON BRANCH perf/cache-getfullargspec

branch: perf/cache-getfullargspec

CHANGE BRANCH

x

Reset

• perf/cache-getfullargspec
• 0.5.0
• 0.6.0
• 0.6.1
• 0.6.2
• 0.6.3
• 0.6.4
• 0.6.5
• 0.7.0
• 0.7.1
• 0.7.2
• 0.7.3
• 0.7.4
• 0.7.5
• 0.7.6
• 0.7.7
• 0.8.1
• 0.8.2
• 0.9.0
• 1.0.0
• 1.0.1
• 2.0.0
• 2.0.0b1
• 2.0.1
• chain
• chore/cleanup-toposort
• chore/remove-dead-code
• chore/tooling-coverage-mypy
• chore/uv-ruff-mypy-migration
• deprecation-warnings
• develop
• docs
• docs/functional-properties-check-keys
• docs/v5-design
• docs/zensical-site
• feat/check-unused-declared-keys
• feat/declared-keys-contract-check
• feat/declared-keys-output-projection
• feat/key-formatter
• feat/named-variadic
• feat/to-dict-typing
• feat/typed-keys-children
• feat/typed-keys-poc
• feat/typed-query-overloads
• feat/unify-key-param
• feature/introspector
• feature/upgrade-dependencies
• fix/chain-sort-before-groupby
• fix/declared-keys-skip-private
• fix/key-reject-structured-types
• fix/match-hash-stable
• main
• markers
• master
• perf/extend-safe-set
• py35
• python-3.12-support
• python-3.6
• python2.6
• refactor/chainpart-typing
• refactor/decouple-chain-builder
• refactor/pattern-dry
• regex-fixes
• rules
• semantic-release
• toposort
• v0.0.1
• v0.1.0
• v0.2.0
• v0.2.1
• v0.3.0
• v0.4.0
• v0.4.1
• wip

Build # 28641828000

Build Type

Pull #78

github

Committed by Toilal

Commit Message

perf: cache getfullargspec on the hot matching path

getfullargspec is called for every validator/formatter/conflict-solver
invocation via loose.call. Its result depends only on the callable, and
those callables are stable singletons for a Rebulk instance's lifetime,
so the introspection is fully redundant across matching calls.

Wrap it in an unbounded lru_cache: ~11% faster repeated parses for a
one-line change. The set of distinct callables is small and bounded,
callables are hashable, and getfullargspec is a pure function of its
argument, so caching by identity cannot change behaviour.

closes #77

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ah1tBeqoVa8sM2CP5ibSJQ

Pull Request Pull Request #78: perf: cache getfullargspec on the hot matching path

Coverage Stats

4 of 4 new or added lines in 1 file covered. (100.0%)

4 existing lines in 2 files now uncovered.

1913 of 1948 relevant lines covered (98.2%)

6.86 hits per line