rust/tantivy
1
0
Fork 0
mirror of https://github.com/quickwit-oss/tantivy.git synced 2026-06-03 17:10:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

SegmentUpdater.add_segment does not need to return true (#693)

Browse Source
This commit is contained in:
Paul Masurel
2019-11-09 21:18:51 +09:00
committed by GitHub
parent 6e4fdfd4bf
commit 88fd7f091a
2 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/indexer/index_writer.rs
View File

@@ -227,7 +227,8 @@ fn index_documents(
delete_cursor,
delete_bitset_opt,
);
Ok(segment_updater.add_segment(segment_entry))
segment_updater.add_segment(segment_entry);
Ok(true)
}
fn apply_deletes(

4
src/indexer/segment_updater.rs
View File

@@ -199,14 +199,12 @@ impl SegmentUpdater {
self.0.pool.spawn_fn(move || Ok(f(me_clone)))
}
pub fn add_segment(&self, segment_entry: SegmentEntry) -> bool {
pub fn add_segment(&self, segment_entry: SegmentEntry) {
self.run_async(|segment_updater| {
segment_updater.0.segment_manager.add_segment(segment_entry);
segment_updater.consider_merge_options();
true
})
.forget();
true
}
/// Orders `SegmentManager` to remove all segments