priyank-p / level-uplevel Goto Github PK

Currently the class that is exported is named LevelDB and our test uses name LevelDB to. Since now the module is called level-uplevel it make sense to use Uplevel as name.

It makes sense to programmatically allow deletion of table, using LevelDB.del

So currently, and most frequently on node 8 test suite, sometime test fails, because the DB is not yet created before the assertion is called. This is because travis infra if possible quite slow.

So currently both, methods, overwrites the InternalProps and ready them, thus effectively wasting 50ms, and losing old data which is critical.

Here is the new draft:

• The createTable should just return if table exist in InternalProps and emit a process.warning.
• For addField same, as create table but this time, throw a error.

And update the readme accordingly.

• If the field is of type String it needs, the current min/max option does not work for it.
• Currently we provide a way to use default value, but for date this does not work we need a new option called timestamp that uses current date.