search

USACE / cwms-data-api

Corps Water Management System RESTful Data Service
MIT License
11 stars 14 forks source link

Added fetchSize to Location getAll query. #793

Closed rma-rripken closed 2 months ago

rma-rripken commented 2 months ago

Suggested by @krowvin in post-CURG feedback. Possible 40% speed improvement on a synthetic test.

rma-rripken commented 2 months ago

With this change I'm seeing getAll with 1000 locations taking 600 ms. About 60% of that time is in db fetch and 40% in JSON serialization. Its not obvious to me how to make the JSON serialization faster. image