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.

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

[Maps] time-based mvt data is refreshed frequently for .mvt scaling-type #79690

Closed
thomasneirynck opened this issue Oct 6, 2020 · 3 comments · Fixed by #93531
Closed

[Maps] time-based mvt data is refreshed frequently for .mvt scaling-type #79690

thomasneirynck opened this issue Oct 6, 2020 · 3 comments · Fixed by #93531
Assignees
Labels
bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation discuss

Comments

@thomasneirynck
Copy link
Contributor

thomasneirynck commented Oct 6, 2020

When scaling-type is .mvt for an index-pattern with time-based data, it is constantly refreshed under panning/zooming when the time-frame is relative. This causes a flash on the map, because the underyling mb-source and mb-layers are constantly being removed and readded.

It's unclear how to handle this. While the behavior looks buggy, it is considered accurate since Maps is trying to show the accurate time-based filter.

Should relative time-ranges be ignored when panning/zooming when scaling-type is .mvt?

@thomasneirynck thomasneirynck added bug Fixes for quality problems that affect the customer experience discuss [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation enhancement New value added to drive a business result labels Oct 6, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@thomasneirynck
Copy link
Contributor Author

Investigate if mapbox/mapbox-gl-js#8048 would fix this

@thomasneirynck thomasneirynck self-assigned this Oct 8, 2020
@thomasneirynck thomasneirynck removed the enhancement New value added to drive a business result label Oct 8, 2020
@thomasneirynck
Copy link
Contributor Author

After discussion with @nreese.

Will first address the bug of comparing absolute timestamps (which is a consequence of just comparing raw urls iso of the SearchFilters). This will fix most FOUC for time-based data, at the expense of possibly showing some delayed data (due to client-side caching).

Whether previous tiles should actively be discarded (e..g with mapbox/mapbox-gl-js#8048, although not yet available as of 1.12) should be considered a different issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation discuss
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants