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

Levels JSONv2 issue #586

Closed

rma-rripken opened this issue Mar 15, 2024 · 1 comment

Labels

approved-W192HQ23F0232-task4

Collaborator

rma-rripken commented Mar 15, 2024

Summary:

User reported issue with Levels
https://discourse.hecdev.net/t/get-location-level-json-error/3339

User was not specifying format meaning that I think the default v1 format (tab) was coming back.

User instructed to try "application/json;version=2"

Selecting “application/json;version=2” actually made it worse. Now I get no response at all.

I looked into why JSONv2 wasn't working. It looks like the jOOQ mapper was duplicating the entire level for each point leading to huge responses.

rma-rripken mentioned this issue

The code was previously building a new LocationLevel for each level v… #574

Merged

MikeNeilson added the approved-W192HQ23F0232-task4 label

Collaborator Author

rma-rripken commented May 17, 2024

#574 Fixed the major performance bug. There still may be other issues but it shouldn't crash quite as quickly

rma-rripken closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment