Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

When adding a big list of layers from external url service, it crashes HSL app. #1886

Closed
DailisLangovskis opened this issue May 26, 2021 · 2 comments · Fixed by #2041
Closed

When adding a big list of layers from external url service, it crashes HSL app. #1886

DailisLangovskis opened this issue May 26, 2021 · 2 comments · Fixed by #2041
Labels
bug performance Performance issues
Milestone
6.0

Comments

@DailisLangovskis
Copy link
Collaborator

Need to expect this get capabilities error, when loading layers. Either forbid user to add so many layers or cancel the request on error fired.
@DailisLangovskis DailisLangovskis added bug performance Performance issues labels May 26, 2021
@jmacura
Copy link
Collaborator

jmacura commented May 26, 2021

relates to #1870 ?

@DailisLangovskis
Copy link
Collaborator Author

relates to #1870 ?

Could be. Only You were mentioning adding a endpoint with many services, that crashes the app. Here I crashed the app, when loading of url services was done, had a list of all layers in datasources url tab, respectively. And only when I tried to add them all to map, the HSL app crashed.

@DailisLangovskis DailisLangovskis added this to the Future milestone Jul 14, 2021
@DailisLangovskis DailisLangovskis linked a pull request Aug 2, 2021 that will close this issue
Issue 1886 - Queues for background capabilities retrieval #2041
Merged
raitisbe added a commit that referenced this issue Aug 7, 2021
@raitisbe
refactor: Create HsQuesModule for managing queues
b42bb11 
ref #1886
raitisbe added a commit that referenced this issue Aug 10, 2021
@raitisbe
refactor: Create HsQuesModule for managing queues
90bde9f 
ref #1886
raitisbe added a commit that referenced this issue Aug 10, 2021
@raitisbe
feat: Add queue library to npm peer dependencies
58954f8 
Queue lib is used to manage paralel requests for wms GetCapabilities and legends

BREAKING CHANGE: `npm i queue`

fix #1886
@jmacura jmacura modified the milestones: Future, 6.0 Aug 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug performance Performance issues
Projects
None yet
Milestone
6.0
Development

Successfully merging a pull request may close this issue.

Issue 1886 - Queues for background capabilities retrieval hslayers/hslayers-ng
2 participants
@jmacura @DailisLangovskis