Skip to content

Commit

Permalink
Merge branch 'merge-pr35'
Browse files Browse the repository at this point in the history
  • Loading branch information
twmht committed Feb 1, 2019
2 parents 0fbacaf + 0c809e1 commit 209ba91
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
4 changes: 1 addition & 3 deletions rocksdb/_rocksdb.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -657,8 +657,7 @@ cdef class PlainTableFactory(PyTableFactory):
index_sparseness=10,
huge_page_tlb_size=0,
encoding_type='plain',
py_bool full_scan_mode=False,
py_bool store_index_in_file=False):
py_bool full_scan_mode=False):

cdef table_factory.PlainTableOptions table_options

Expand All @@ -676,7 +675,6 @@ cdef class PlainTableFactory(PyTableFactory):
raise ValueError("Unknown encoding_type: %s" % encoding_type)

table_options.full_scan_mode = full_scan_mode
table_options.store_index_in_file = store_index_in_file

self.factory.reset( table_factory.NewPlainTableFactory(table_options))
#############################################
Expand Down
3 changes: 2 additions & 1 deletion rocksdb/tests/test_options.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,8 @@ class TestOptions(unittest.TestCase):
def test_compaction_pri(self):
opts = rocksdb.Options()
# default compaction_pri
self.assertEqual(opts.compaction_pri, rocksdb.CompactionPri.by_compensated_size)
# self.assertEqual(opts.compaction_pri, rocksdb.CompactionPri.by_compensated_size)
self.assertEqual(opts.compaction_pri, rocksdb.CompactionPri.min_overlapping_ratio)
opts.compaction_pri = rocksdb.CompactionPri.by_compensated_size
self.assertEqual(opts.compaction_pri, rocksdb.CompactionPri.by_compensated_size)
opts.compaction_pri = rocksdb.CompactionPri.oldest_largest_seq_first
Expand Down

0 comments on commit 209ba91

Please sign in to comment.