-
Notifications
You must be signed in to change notification settings - Fork 477
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
Have you done any benchmark on it compared to others? #6
Comments
Hi there, we did benchmark lokijs with similar products, namely PouchDB, NeDB and On Sun, Oct 12, 2014 at 6:48 PM, vegarringdal notifications@github.com
|
Atm I'm just testing to see if I understand it, testing it on server side on Wakanda in a module. Looking at find and findOne, would it not be easier to use query word for find, and just use find instead of find one. |
All great suggestions! As for the using the array for insert, it was a simple change so I applied On Sun, Oct 12, 2014 at 8:26 PM, vegarringdal notifications@github.com
|
Maybe you should consider adding "stamp" also to every entity in a collection. |
Hi, thanks a lot for the feedback, it is precious. We are considering revisions, similar to what couchdb has in place, also On Sun, Oct 12, 2014 at 11:28 PM, vegarringdal notifications@github.com
|
hmm, I see the issue. (the insert array issue worked fine, thx) Well for the sync of the database you will need a way to lock it. Just thinking out laud, you prob have a better way of solving this. Database: lockedState: (attribute) exportOfflineCopy: (function) syncOfflineCopy(function(localCopy) Entity: _localModified: (attribute, bool) _localNew:(attribute, bool) _stamp: (attribute, num/int) Issues, should a local copy of database have a expire date/time stamp, to avoid to old data? Should a local copy be able to add new collections, making sync more complicated. How do you want to solve stamp issues? this simple suggestion/ ideas got long now, so better stop |
Hi, lots of food for thought there! The two major concerns are db size and performance, in that these two On Mon, Oct 13, 2014 at 7:12 PM, vegarringdal notifications@github.com
|
would lamport timestamps help here? btw: interesting project. Kudos! |
very interesting. It would definitely help in a distributed version of On Wed, Nov 5, 2014 at 1:33 AM, Geert-Jan Brits notifications@github.com
|
Closing the issue as benchmark are in the source tree, i'm opening a lamport timestamps issue separate to this. |
Have you done any benchmark on it compared to others?
Thx, will give this a try on wakanda server (wakanda.org)
The text was updated successfully, but these errors were encountered: