Skip to content

Reduce connection pool borrows in menus, metadata loading, lineage, NAb, and audit grids - #8130

Open
labkey-jeckels wants to merge 7 commits into
developfrom
fb_reduceConnectionGetting
Open

labkey-jeckels wants to merge 7 commits into
developfrom
fb_reduceConnectionGetting

Conversation

@labkey-jeckels

@labkey-jeckels labkey-jeckels commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor

Rationale

Cut the overhead of repeatedly borrowing and returning pooled connections on common request paths. For example, the product menu API averaged 17 borrows per call in TeamCity runs and should drop to 3–5 with the companion limsModules change, and loading a table's schema metadata drops from 3 borrows to 1.

Related Pull Requests

Changes

  • Product menu sections load their items under one held connection instead of one borrow per lazy query
  • Schema metadata loading shares one connection across its column, primary key, and index passes
  • Plate and dilution data for NAb runs load per well group in batches rather than per well
  • Lineage queries and data type exclusions are cached or batched
  • Domain invalidation and object deletes clear only the affected ontology cache entries instead of everything
  • Audit grid columns memoize per-row lookups
  • Query renames clear ontology caches so lookups rewritten in other domains are picked up
  • Plates cache their run container by id so cached plates pass the cache safety check

Tasks

  • Claude Code Review
  • Manual Testing - N/A
  • Test Automation

TeamCity: BVT, Daily, Biologics, Sample Manager, LIMS, Inventory, nab, and endpoint suites pass on this branch.

…neage, NAb, and audit grids

Cache data type exclusions, batch per-section and per-well-group queries, share one held connection across repeated statements, narrow invalidateDomain and object-delete property cache clears, and memoize per-row audit lookups.
Object deletes remove exact property map keys and run one deduped container filter only when unknown URIs were deleted, exclusion lookups cache only real containers, and NAb sample maps are published fully built.
@labkey-jeckels
labkey-jeckels requested a review from a team October 4, 2026 22:00
@labkey-jeckels labkey-jeckels self-assigned this Oct 4, 2026
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