Details
-
Bug
-
Status: Closed (View Workflow)
-
P2
-
Resolution: Done
-
None
-
None
-
Q3 scope, Gulfstream Sprint 95, Gulfstream Sprint 96, Gulfstream Sprint 97
-
8
-
Gulfstream
Description
Overview:
Steps to Reproduce:
Log into performance environment FOLIO environment as folio user
Make GET rtac/instanceId where each instance has more than 300 holdings record
Expected Results: getRtacById method is O(n^3)
Actual Results: getRtacById is O( n )
Additional Information:
The main issue is with https://github.com/folio-org/mod-rtac/blob/d8e51a8a82b69a6f1717b9c7abf60342ded99246/src/main/java/org/folio/rest/impl/RtacResourceImpl.java#L48
Even after leveraging vertx's asynchronous behavior and considering the fact that 1 instance -> many holdings -> each holding -> many items -> each item -> many loans, this call chain triggers Big O(n^3) or Big O(holdings * items * loans) algorithmic complexity.
https://issues.folio.org/browse/PERF-61
URL:
https://wiki.folio.org/pages/viewpage.action?pageId=36579550
The goal is to get RTAC execution to below 5 seconds for more than 300 holdings.
Interested parties:
All consumers of mod-rtac
TestRail: Results
Attachments
Issue Links
- blocks
-
MODRTAC-37 Better handle periodicals in response
-
- Closed
-
- defines
-
UXPROD-2425 RTAC | Holdings - EDGE-RTAC and MOD-RTAC API updates
-
- Closed
-
- is blocked by
-
MODINVSTOR-518 Transform OAI-PMH view API in Inventory into general API
-
- Closed
-
- is required by
-
EDGRTAC-29 REST batching support
-
- Closed
-
- relates to
-
MODINVSTOR-614 inventory-storage crashes after grabbing all available DB connections
-
- Closed
-
-
MODRTAC-34 REST batching support
-
- Closed
-
-
MODRTAC-40 [SPIKE] Measurement of time penalty for additional proxification
-
- Closed
-
-
EDGRTAC-33 Memory leak of ~200mb for /rtac?instanceIds=id1,id2,idn API
-
- Closed
-
-
MODINVSTOR-534 Create database view for RTAC performance optimization
-
- Closed
-
-
MODRTAC-48 Issues in batch API when querying an instance with more than 50 holdings
-
- Closed
-
-
MODRTAC-53 Folio facade mapping error - 504
-
- Open
-
-
MODRTAC-64 RTAC-batch: Wrong response for not-existing instance
-
- Closed
-
-
PERF-61 Rerun RTAC tests
-
- Closed
-
- requires
-
PERF-117 Test mod-rtac (Honeysuckle)
-
- Closed
-
- mentioned in
-
Page Loading...
1.
|
Switch to POST /inventory-hierarchy/items-and-holdings endpoint for getting items |
|
Closed | Unassigned |