Page MenuHomePhabricator

Consolidate more Ferret engine code into FerretEngine
ClosedPublic

Authored by epriestley on Sep 5 2017, 4:30 PM.
Tags
None
Referenced Files
F19510360: D18533.id44499.diff
Sun, Jan 11, 8:15 AM
F19080814: D18533.id44499.diff
Dec 2 2025, 2:31 AM
F18964370: D18533.id44499.diff
Nov 14 2025, 7:31 AM
F18912147: D18533.diff
Nov 9 2025, 4:42 AM
F18863400: D18533.id.diff
Nov 2 2025, 11:09 PM
F18857961: D18533.diff
Nov 1 2025, 7:10 PM
F18786057: D18533.id.diff
Oct 14 2025, 4:55 PM
F18768353: D18533.id44512.diff
Oct 8 2025, 3:10 AM
Subscribers
None

Details

Summary

Ref T12819. Earlier I separated some ngram code into an "ngram engine" hoping to share it across the simple Ngrams stuff and the full Ferret stuff, but they actually use slightly different rules. Just pull more of this stuff into FerretEngine to reduce the number of moving pieces and the amount of code duplication.

Test Plan

Searched for terms, rebuilt indexes.

Diff Detail

Repository
rP Phabricator
Branch
ferret1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18283
Build 24601: Run Core Tests
Build 24600: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Sep 5 2017, 5:15 PM
This revision was automatically updated to reflect the committed changes.