-
Notifications
You must be signed in to change notification settings - Fork 46
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.
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
Have a separate cache for the OGC servers #9911
Conversation
7eb16aa
to
1e2f030
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems really difficult to not depends on theme_view in admin.
But this will block us it at some time we want to put admin in another container.
It might be interesting having some tests, available for discussion on this. |
1e2f030
to
0b6cc28
Compare
Apparently this PR still contains some anti-patterns that I would like to resolve before merge. |
ae2309d
to
1d10e98
Compare
@arnaud-morvan ready for a final review :-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems the view to refresh OGC Server cache is missing.
Feel free to merge after fix.
1d10e98
to
032f4f9
Compare
032f4f9
to
c312543
Compare
Tested on demo, at home throw VPN on my slow new computer
GSLUX-460
Before:
First: 33.6s
Second: 0.7s
After Layer change: 34.6s
After OGC server change: 32.4s
After:
First: 43.6s
Second: 0.9s
After Layer change: 4.1s
After OGC server change: 4.1s (But we should wait that the background task ended, about 20s)