Skip to content

BDMS-776: Increased the export hook pageSize from 20 to 500 on well list page#277

Merged
TylerAdamMartinez merged 2 commits into
stagingfrom
TAM-BDMS-776
May 20, 2026
Merged

BDMS-776: Increased the export hook pageSize from 20 to 500 on well list page#277
TylerAdamMartinez merged 2 commits into
stagingfrom
TAM-BDMS-776

Conversation

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem/context:

  • The export feature was making many paginated requests to the backend using a default pageSize of 20.
  • Large exports required going through ~69 pages to generate a single CSV export.
  • This resulted in slower export times due to the repeated frontend ↔ backend round-trip and request overhead.

How

Implementation summary - the following was changed/added/removed:

  • Increased the export hook pageSize from 20 to 500.
  • This reduces the total number of requests required during export generation.
  • Fewer requests improve performance by reducing:
    • HTTP request overhead
    • frontend processing time
    • backend pagination overhead
    • repeated serialization/deserialization work
  • Local speed test results:
    • pageSize: 20 → 8.21s
    • pageSize: 500 → 3.53s

Notes

Any special considerations, workarounds, or follow-up work to note?

  • A larger pageSize improves performance for the current implementation, but the export process still relies on multiple paginated API requests from the frontend.
  • For additional performance improvements, we could implement a dedicated backend export endpoint that performs the export in a single request

@TylerAdamMartinez TylerAdamMartinez self-assigned this May 18, 2026
@TylerAdamMartinez TylerAdamMartinez added the enhancement New feature or request label May 18, 2026
@TylerAdamMartinez
Copy link
Copy Markdown
Contributor Author

Does the export need to exactly mirror the currently filtered/paginated list view?

@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-tam-bdms-776-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor Author

Does the export need to exactly mirror the currently filtered/paginated list view?

@jeremyzilar This PR now includes the same search parameters as the search, so the exported sheet will return the same search results

@github-actions
Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-tam-bdms-776-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@TylerAdamMartinez TylerAdamMartinez merged commit 3a8d5ef into staging May 20, 2026
3 of 4 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the TAM-BDMS-776 branch May 20, 2026 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants