search

t44k / clj-rocksdb

Eclipse Public License 1.0
16 stars 3 forks source link

Sync doesn't work #1

Closed cordawyn closed 4 years ago

cordawyn commented 4 years ago

Hi!

It seems that this https://github.com/kotyo/clj-rocksdb/blob/master/src/clj_rocksdb.clj#L335 should call .setSync (setter method) instead of .sync (getter method). (You can check with https://github.com/facebook/rocksdb/blob/master/java/src/main/java/org/rocksdb/WriteOptions.java#L64)

Using clj-rocksdb/sync db crashes for me because of this.

t44k commented 4 years ago

:+1: Thank you for the comment. It is fixed now.