Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Update layers/tiles while displaying #2481

Closed
1 task done
kkaefer opened this issue Oct 1, 2015 · 5 comments
Closed
1 task done

Update layers/tiles while displaying #2481

kkaefer opened this issue Oct 1, 2015 · 5 comments

Comments

@kkaefer
Copy link
Member

kkaefer commented Oct 1, 2015

We should support updating tiles/layers while they are being displayed. This can happen when the tile data expires while it is still being displayed. It's also a prerequisite for #2445.

Prerequisites:

Likely obstacles:

  • Figure out how to safely swap out buffers
  • Tile data lifetimes
  • Should API work on a per-layer or a per-tile level? Maybe both?
@willwhite
Copy link

@kkaefer what is your latest thinking around how to decide when to refresh the map? Will this be triggered manually or could we keep track of all tiles that have an Expires header and force a refresh when that date is hit? This would let the server decide when/if data should be refreshed.

@kkaefer
Copy link
Member Author

kkaefer commented Oct 12, 2015

The idea of this feature is to respect the Expires/max-age headers the server sends. We already do that today, but we never update a tile after it is being displayed. That means, that in situations where we display tiles or maps for a very long period of time, like in visualizations, we will continue to show a tile despite it being outdated. So ultimately, when we display a tile, we will also set a timer that will trigger a refresh once the server sent expires header is done.

@willwhite
Copy link

Gotcha, will it do the same for max-age or will the auto-refresh behavior be limited to expires?

@kkaefer
Copy link
Member Author

kkaefer commented Oct 13, 2015

On the client side, we calculate an absolute date when receiving the data, Expires will be taken verbatim; max-age will be added to the current time. After that, we don't discriminate between those two headers.

@kkaefer
Copy link
Member Author

kkaefer commented Oct 21, 2015

Subsumed by #2617.

@kkaefer kkaefer closed this as completed Oct 21, 2015
@incanus incanus added the P0 label Nov 4, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants