Page MenuHomePhabricator

Remove some redundant information from the Ferret engine index
ClosedPublic

Authored by epriestley on Sep 8 2017, 3:16 PM.
Tags
None
Referenced Files
F19100271: D18580.id.diff
Dec 4 2025, 10:59 PM
F19090316: D18580.diff
Dec 3 2025, 11:10 AM
F18863419: D18580.id.diff
Nov 2 2025, 11:14 PM
F18856716: D18580.diff
Nov 1 2025, 11:14 AM
F18855983: D18580.id44620.diff
Nov 1 2025, 7:33 AM
F18789069: D18580.id44617.diff
Oct 15 2025, 7:49 AM
F18783215: D18580.diff
Oct 13 2025, 4:53 AM
F18733291: D18580.id44617.diff
Sep 30 2025, 10:25 PM
Subscribers
None

Details

Summary

Ref T12819. The "full" field has all other fields, and the "core" field has "title" and "body". Due to the way the "full" and "core" fields were being built, the "core" field also got included in the "full" field, so the "full" field has two copies of the title, two copies of the body, and then one copy of everything else.

Put only one copy of each distinct thing in each "full" and "core". Also, simplify the logic a little bit so we build these virtual fields in a more consistent way.

Test Plan

Ran bin/search index and looked at the fields in the database, saw less redundant information.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable