Page MenuHomePhabricator

Add ferret support to Slowvote
AcceptedPublic

Authored by amckinley on Jul 3 2019, 11:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 14, 5:26 PM
Unknown Object (File)
Feb 5 2024, 5:50 PM
Unknown Object (File)
Feb 3 2024, 10:50 PM
Unknown Object (File)
Feb 1 2024, 7:12 AM
Unknown Object (File)
Dec 22 2023, 9:08 AM
Unknown Object (File)
Dec 22 2023, 2:38 AM
Unknown Object (File)
Dec 22 2023, 2:38 AM
Unknown Object (File)
Dec 19 2023, 11:07 PM
Subscribers
Restricted Owners Package

Details

Reviewers
epriestley
Summary

Keep pushing on PHI1292. AFAICT, there are only four classes that support Spaces but lack fulltext support:

  • PhabricatorMetaMTAApplicationEmail
  • PhabricatorPaste
  • PhabricatorPhurlURL
  • PhabricatorSlowvotePoll

These all look easy, so just pound out the first one of these.

Test Plan
  • bin/storage upgrade
  • bin/seach index --type POLL
  • Looked at DB tables, ran some searches, confirmed search-with-Space worked.

Diff Detail

Repository
rP Phabricator
Branch
poll-ferret (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 23110
Build 31730: Run Core Tests
Build 31729: arc lint + arc unit

Event Timeline

Owners added a subscriber: Restricted Owners Package.Jul 3 2019, 11:16 PM

Ah, nice!

You can also follow 20190412.herald.01.rebuild.php to add a migration which queues a worker to queue other workers to rebuild the index now. It's probably reasonable to do for Slowvote since installs are unlikely to have millions upon millions of these. But we could also do it later.

This revision is now accepted and ready to land.Jul 3 2019, 11:20 PM

But we could also do it later.

Yup, I was planning on adding one at the end these four revs.