Repository navigation
Cache NAb runs per run and fit type - #1079
Merged
Merged
Conversation
Graph images and details for the same run and curve fit now share one built run, and high-throughput plates load their stored DilutionData in one query.
3 tasks done
cnathe
reviewed
Oct 7, 2026
cnathe
approved these changes
Oct 7, 2026
labkey-jeckels
added a commit
to LabKey/platform
that referenced
this pull request
Oct 9, 2026
…Ab, and audit grids (#8130) ## 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 - LabKey/commonAssays: LabKey/commonAssays#1079 - LabKey/limsModules: LabKey/limsModules#2515 - LabKey/testAutomation: LabKey/testAutomation#3236 ## 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 - [x] Claude Code Review - [x] Manual Testing - N/A - [x] Test Automation TeamCity: BVT, Daily, Biologics, Sample Manager, LIMS, Inventory, nab, and endpoint suites pass on this branch.
labkey-jeckels
pushed a commit
to LabKey/testAutomation
that referenced
this pull request
Oct 9, 2026
## Rationale See related commonAssays PR for rationale. This PR adds a test that NAb QC exclusions reach other sessions' cached runs. ## Related Pull Requests - LabKey/platform#8130 - LabKey/limsModules#2515 - LabKey/commonAssays#1079 ## Changes - Saves well exclusions from a separate API session and checks the browser session sees the recalculated results, for both an explicit and the default curve fit.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rationale
Cut the overhead of repeated connection borrows when rendering NAb results. For example, each NAb graph image averaged 164 borrows in TeamCity runs because it rebuilt the whole run; keying the run cache on run and fit type brings a cache hit down to about 3.
Related Pull Requests
Changes
TeamCity: nab Postgres passes on this branch.