search

brave / sync

deprecated Brave sync server. (sync now uses a fork of the Chromium sync protocol.)
Mozilla Public License 2.0
204 stars 41 forks source link

Compaction follow-up: remedy for out of sync local/remote records #351

Closed darkdh closed 5 years ago

darkdh commented 5 years ago

per discussion with @bridiver on slack

We should make the most of list requests which are expensive. Except compaction, we should also be checking local vs remote records and filling in anything that is missing on either end, all based on most recent timestamp

darkdh commented 5 years ago

It will be remote->local comparison. local->remote will be addressed in https://github.com/brave/brave-browser/issues/6674 because we need all bookmarks from local client and if client has tremendous amount of bookmarks, that will overwhelm the device memory.