Page MenuHomePhabricator
Differential D12432

T7827: Boost Elasticsearch results on title match.
ClosedPublic
Actions

Authored by Pawka on Apr 16 2015, 7:36 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 3, 5:04 PM
Unknown Object (File)
Mon, Apr 1, 9:57 AM
Unknown Object (File)
Sat, Mar 30, 5:23 PM
Unknown Object (File)
Thu, Mar 28, 5:59 PM
Unknown Object (File)
Mar 24 2024, 8:37 PM
Unknown Object (File)
Mar 22 2024, 7:01 PM
Unknown Object (File)
Mar 20 2024, 9:49 PM
Unknown Object (File)
Mar 16 2024, 6:32 PM
View All Files
Subscribers
epriestley
Korvin
WikiChad

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T7827: Boost Elasticsearch results if match is found on title instead of body.
Commits
Restricted Diffusion Commit
rP3a17220ce3da: T7827: Boost Elasticsearch results on title match.
Summary

If match is found on document title, boost result.

Test Plan
  • Setup phabricator with elasticsearch.
  • Index documents with bin/search index --all
  • Use search and expect query matching titles appear on the top of results list.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Pawka updated this revision to Diff 29843.Apr 16 2015, 7:36 AM
Pawka added a task: T7827: Boost Elasticsearch results if match is found on title instead of body..
Pawka retitled this revision from to T7827: Boost Elasticsearch results on title match..
Pawka updated this object.
Pawka edited the test plan for this revision. (Show Details)
Pawka added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 16 2015, 7:36 AM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley edited edge metadata.Apr 16 2015, 2:38 PM
epriestley added a subscriber: WikiChad.

@WikiChad, do you have thoughts on this? It seems reasonable to me, but I'm not familiar enough with ElasticSearch to be able to catch any gotchas.

WikiChad added a comment.Apr 16 2015, 2:42 PM

This should work fine. boolean "should" queries do basically exactly what you'd expect.

epriestley accepted this revision.Apr 16 2015, 2:42 PM
epriestley edited edge metadata.

Awesome, thanks for taking a look!

This revision is now accepted and ready to land.Apr 16 2015, 2:42 PM
Closed by commit rP3a17220ce3da: T7827: Boost Elasticsearch results on title match. (authored by Povilas Balzaravicius Pawka <povilas@uber.com>, committed by epriestley). · Explain WhyApr 16 2015, 2:43 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
search/
engine/
11 lines

Diff 29844

View Options

src/applications/search/engine/PhabricatorSearchEngineElastic.php

Loading...
Phabricator · Built by Phacility