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

Sync chunks since last session #5

Closed
Gjum opened this issue May 18, 2022 · 1 comment · Fixed by #11 or #21
Closed

Sync chunks since last session #5

Gjum opened this issue May 18, 2022 · 1 comment · Fixed by #11 or #21
Assignees

Comments

@Gjum
Copy link
Member

Gjum commented May 18, 2022

  • on connect, server sends list of each (up to ~1260) region's newest chunk ts - so we can ignore regions that are fully synced
  • client finds region where its oldest chunk ts is older than server's newest chunk ts in that region
    • use file modification date? so we don't have to read the file's contents. must set explicitly because by default it's the newest chunk ts
  • if a region is too old, client sends that region's chunk ts's (1024) to server; server responds with all newer chunk data of that region
  • client only requests regions when not busy rendering
@Gjum Gjum added this to MapSync May 18, 2022
@Gjum Gjum moved this to In Progress in MapSync May 20, 2022
This was referenced May 24, 2022
@Gjum Gjum moved this from In Progress to In Review in MapSync May 25, 2022
@Gjum Gjum closed this as completed in #11 May 25, 2022
Repository owner moved this from In Review to Done in MapSync May 25, 2022
@Gjum Gjum moved this from Done to In Review in MapSync May 26, 2022
@Gjum Gjum moved this from In Review to Done in MapSync May 26, 2022
@Gjum
Copy link
Member Author

Gjum commented May 26, 2022

reopening so we can implement this more intricate protocol

the reason is that the current protocol (everything after last connected timestamp) does not allow for stopping and resuming catchup when leaving the server or changing dimensions #17

@Gjum Gjum reopened this May 26, 2022
@Gjum Gjum moved this from Done to In Progress in MapSync May 26, 2022
@Gjum Gjum moved this from In Progress to In Review in MapSync May 30, 2022
@Gjum Gjum mentioned this issue May 30, 2022
@Gjum Gjum closed this as completed in #21 Jun 1, 2022
Repository owner moved this from In Review to Done in MapSync Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants