neumino / reqlite

Reqlite - RethinkDB in JavaScript
MIT License
338 stars 25 forks source link

Documentation of Missing Features. #3

Closed zephraph closed 9 years ago

zephraph commented 9 years ago

I'm going to go ahead and start a list of features I find that are missing from reql. This will be updated as we go. Please let me know if I missed anything!

Stream

Not Implemented

r (rethink object, may not be desired)

neumino commented 9 years ago

Thanks for putting that up @zephraph.

So I have recently started to work again on reqlite. I'm sorting out a few things first. but I should be clear to push in 2-3 weeks. I don't remember all the details, but I can provide a list of things that are currently not available in my branch

Stuff that I should be able to finish for next release

Things I would like to do

Things I'm not sure

Things I don't plan to do (but I would merge a PR if someone does it)

As far as I know, all the rest work. There might be a few assertions missing for types, but that's all. The test framwork was reworked a bit. The test are basically now running queries against reqlite and rethinkdb and compare the output.

zephraph commented 9 years ago

/cc @Slava

@neumino, I'm really wanting to see baked in support for rethinkdb in meteor so I'll do everything that I can to facilitate this library. Feel free to assign me tasks if you would like me to investigate or work on certain tasks.

Slava commented 9 years ago

I think we need more tests on the queries. At this point I am not even sure if basic operations like "add" and "orderBy" work w/o bugs. Fixing the existing tests would be a good start. Also, @neumino, are there tests we can adopt from the main rethinkdb repo?

neumino commented 9 years ago

@Slava -- Yes, for now I wrote the queries myself to make sure that all the code paths were tested. I plan to just pull all the tests for the JS driver and dump it in reqlite.

neumino commented 9 years ago

Opened a lot of issues to track done what's left. Closing this meta issue.

neumino commented 9 years ago

@Slava -- Could you check that _saveOriginal still works for your need? It was a big merge ;-)

neumino commented 9 years ago

@zephraph -- if you want to help now, there are tons of things to grab.

zephraph commented 9 years ago

I see that, haha. I'll look through the open tasks.