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

move to leveldb storage #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

vmakhaev
Copy link

Using leveldb on client shows race condition that should be investigated

@gritzko
Copy link
Owner

gritzko commented Oct 19, 2014

(1) the race is because of the LevelStorage.prototype.isRoot flag. Server-side storage needs "true", client-side cache needs "false".
isRoot means "this storage is the root of the replica tree".

@gritzko
Copy link
Owner

gritzko commented Oct 19, 2014

(2) Does it work without the timeout?

@vmakhaev
Copy link
Author

  1. Ok, I will test it
  2. It works, but as far as it thinks that list is empty, it adds new empty item each time you clear client's storage and reload app

@vmakhaev
Copy link
Author

isRoot did not solve the problem. Maybe problem in initialization of Vector.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants